automatic workflow to request the correct item / address of a requester #144

Open
clarity wants to merge 40 commits from clarity/ticket-requests into dev
Owner
No description provided.
clarity requested review from j3d1 2026-09-27 14:16:29 +00:00
clarity self-assigned this 2026-09-27 14:16:35 +00:00
Author
Owner

ich werde das nochmal rebasen - die commits aus 2025 sind quasi nicht mehr relevant für dieses feature.

ich werde das nochmal rebasen - die commits aus 2025 sind quasi nicht mehr relevant für dieses feature.
clarity force-pushed clarity/ticket-requests from 9b84b4068c
All checks were successful
/ frontend (push) Successful in 1m23s
/ test (push) Successful in 2m3s
/ frontend (pull_request) Successful in 1m19s
/ test (pull_request) Successful in 1m56s
to 2efa8cd92a
All checks were successful
/ frontend (pull_request) Successful in 1m26s
/ frontend (push) Successful in 1m26s
/ test (pull_request) Successful in 2m9s
/ test (push) Successful in 2m11s
2026-09-27 14:22:23 +00:00
Compare
added buttons to the related items column
All checks were successful
/ frontend (pull_request) Successful in 1m29s
/ frontend (push) Successful in 1m29s
/ test (push) Successful in 2m5s
/ test (pull_request) Successful in 2m7s
e8156304d5
Author
Owner

hinweis: accessibility checken

hinweis: accessibility checken
Author
Owner

kommentar von jedi: pull request kleiner machen. es werden noch viele dinge angefasst, die man vielleicht nicht anfassen müsste.

kommentar von jedi: pull request kleiner machen. es werden noch viele dinge angefasst, die man vielleicht nicht anfassen müsste.
clarity force-pushed clarity/ticket-requests from e8156304d5
All checks were successful
/ frontend (pull_request) Successful in 1m29s
/ frontend (push) Successful in 1m29s
/ test (push) Successful in 2m5s
/ test (pull_request) Successful in 2m7s
to c9f6e327dc
All checks were successful
/ frontend (push) Successful in 1m33s
/ frontend (pull_request) Successful in 1m35s
/ test (push) Successful in 2m16s
/ test (pull_request) Successful in 2m18s
2026-10-11 14:27:53 +00:00
Compare
clarity changed target branch from testing to dev 2026-10-11 15:01:58 +00:00
clarity force-pushed clarity/ticket-requests from c9f6e327dc
All checks were successful
/ frontend (push) Successful in 1m33s
/ frontend (pull_request) Successful in 1m35s
/ test (push) Successful in 2m16s
/ test (pull_request) Successful in 2m18s
to 62119ae2f4
All checks were successful
/ frontend (pull_request) Successful in 1m31s
/ frontend (push) Successful in 1m35s
/ test (push) Successful in 2m11s
/ test (pull_request) Successful in 2m13s
2026-10-11 15:14:08 +00:00
Compare
@ -42,1 +42,4 @@
# Absolute base of the public site, used to build links that go out by mail.
# Unlike PRIMARY_HOST and MAIL_DOMAIN above this is a full URL, not a bare host.
PUBLIC_BASE_URL = os.getenv('PUBLIC_BASE_URL', f'https://{PRIMARY_HOST}')
Author
Owner

jedi: inline in die email, keine eigene variable

jedi: inline in die email, keine eigene variable
clarity marked this conversation as resolved
@ -0,0 +1,64 @@
"""Response builders for stored files.
Author
Owner

jedi: komplette serving.py inlinen. das muss nicht in eine eigene datei ausgelagert werden.

jedi: komplette serving.py inlinen. das muss nicht in eine eigene datei ausgelagert werden.
clarity marked this conversation as resolved
@ -0,0 +1,80 @@
"""The countries the application knows, keyed by ISO 3166-1 alpha-3.
Author
Owner

jedi: countries einfach vom user als string eingeben lassen.

jedi: countries einfach vom user als string eingeben lassen.
clarity marked this conversation as resolved
@ -189,0 +318,4 @@
ticket_updates.none_matched(self)
class AddressRequest(TicketRequest):
Author
Owner

jedi: Zu komplex. Zu viele Methoden, warum transaction atomic?

jedi: Zu komplex. Zu viele Methoden, warum transaction atomic?
Author
Owner

@j3d1 die ganzen transaction.atomic sind jetzt raus.

@j3d1 die ganzen transaction.atomic sind jetzt raus.
Author
Owner

und die alten methoden zur adress-validierung auch. die kamen noch aus dem shipment-feature; u.a. weil die adressen vor übergabe an den shipment-provider validiert werden mussten.

und die alten methoden zur adress-validierung auch. die kamen noch aus dem shipment-feature; u.a. weil die adressen vor übergabe an den shipment-provider validiert werden mussten.
Author
Owner

die open_for methode ist jetzt in .save() gewandert. Die Logik da drin bewirkt, das ggf. noch bestehende Requests gecancelt werden, wenn jemensch einen neuen Request erzeugt (z.b. mit einer neuen Item-Kombination o.ä.)

die open_for methode ist jetzt in .save() gewandert. Die Logik da drin bewirkt, das ggf. noch bestehende Requests gecancelt werden, wenn jemensch einen neuen Request erzeugt (z.b. mit einer neuen Item-Kombination o.ä.)
Author
Owner

@j3d1 bevor du dir das frontend anschaust, würde ich das auch alles nochmal kleiner machen wollen. gebe dir einen headsup.

@j3d1 bevor du dir das frontend anschaust, würde ich das auch alles nochmal kleiner machen wollen. gebe dir einen headsup.
Trim one-off helpers from the request ui and refresh one ticket after sending
All checks were successful
/ frontend (pull_request) Successful in 1m27s
/ frontend (push) Successful in 1m29s
/ test (pull_request) Successful in 2m10s
/ test (push) Successful in 2m12s
cb18eea5ad
Show request times with toLocaleString like the timeline and drop timeAgo
All checks were successful
/ frontend (pull_request) Successful in 1m27s
/ frontend (push) Successful in 1m29s
/ test (pull_request) Successful in 2m10s
/ test (push) Successful in 2m12s
e6c80ddfde
Show request states as their raw tags and keep the badge colours in the ticket view
All checks were successful
/ frontend (pull_request) Successful in 1m26s
/ frontend (push) Successful in 1m27s
/ test (pull_request) Successful in 2m6s
/ test (push) Successful in 2m9s
81ea997fc5
clarity force-pushed clarity/ticket-requests from 81ea997fc5
All checks were successful
/ frontend (pull_request) Successful in 1m26s
/ frontend (push) Successful in 1m27s
/ test (pull_request) Successful in 2m6s
/ test (push) Successful in 2m9s
to 9d39adc291
All checks were successful
/ frontend (push) Successful in 1m23s
/ frontend (pull_request) Successful in 1m22s
/ test (pull_request) Successful in 1m59s
/ test (push) Successful in 2m2s
2026-10-12 16:31:10 +00:00
Compare
clarity changed title from WIP: automatic workflow to request the correct item / address of a requester to automatic workflow to request the correct item / address of a requester 2026-10-12 16:31:35 +00:00
clarity closed this pull request 2026-10-31 15:15:40 +00:00
Author
Owner

will be rewritten

will be rewritten
clarity reopened this pull request 2026-11-02 20:42:19 +00:00
Remove the tickets README, the repo keeps docs out of the app directories
Some checks failed
/ test (push) Has been cancelled
/ frontend (push) Has been cancelled
/ frontend (pull_request) Successful in 1m23s
/ test (pull_request) Successful in 1m55s
f0316bd337
clarity force-pushed clarity/ticket-requests from f0316bd337
Some checks failed
/ test (push) Has been cancelled
/ frontend (push) Has been cancelled
/ frontend (pull_request) Successful in 1m23s
/ test (pull_request) Successful in 1m55s
to 8e9d1655de
All checks were successful
/ frontend (pull_request) Successful in 1m22s
/ frontend (push) Successful in 1m23s
/ test (pull_request) Successful in 1m53s
/ test (push) Successful in 1m55s
2026-11-02 20:52:39 +00:00
Compare
All checks were successful
/ frontend (pull_request) Successful in 1m22s
/ frontend (push) Successful in 1m23s
/ test (pull_request) Successful in 1m53s
/ test (push) Successful in 1m55s
This pull request has changes conflicting with the target branch.
  • core/tickets/api_v2.py
  • core/tickets/models.py
  • core/tickets/serializers.py
  • core/tickets/tests/v2/test_requests.py
  • core/tickets/tests/v2/test_requests_api.py
  • core/tickets/tests/v2/test_tickets.py
  • web/src/components/RequestLinkModal.vue
  • web/src/components/SelectableItemGrid.vue
  • web/src/components/Timeline.vue
  • web/src/components/TimelineRequest.vue
  • web/src/router.js
  • web/src/store.js
  • web/src/utils.js
  • web/src/views/Ticket.vue
View command line instructions

Manual merge helper

Use this merge commit message when completing the merge manually.

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin 'clarity/ticket-requests:clarity/ticket-requests'
git switch 'clarity/ticket-requests'

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch 'dev'
git merge --no-ff 'clarity/ticket-requests'
git switch 'clarity/ticket-requests'
git rebase 'dev'
git switch 'dev'
git merge --ff-only 'clarity/ticket-requests'
git switch 'clarity/ticket-requests'
git rebase 'dev'
git switch 'dev'
git merge --no-ff 'clarity/ticket-requests'
git switch 'dev'
git merge --squash 'clarity/ticket-requests'
git switch 'dev'
git merge --ff-only 'clarity/ticket-requests'
git switch 'dev'
git merge 'clarity/ticket-requests'
git push origin 'dev'
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
c3lf/c3lf-system-3!144
No description provided.