- add raw message field to request history table, and relevant view to access it
- add email as choice for the request history actor roles, when the message can't be matched to a known actor
Depends on D7798.
Details
Details
added a couple of python tests for this. Testing the new link in
cypress will happen when we hook up email processing
Diff Detail
Diff Detail
- Repository
- rDWAPPS Web applications
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
Event Timeline
Comment Actions
Build is green
Patch application report for D7799 (id=28172)
Could not rebase; Attempt merge onto 84fd757cf8...
Updating 84fd757c..3f4da941 Fast-forward .../bundles/add_forge/add-request-history-item.ejs | 3 + assets/src/bundles/add_forge/request-dashboard.js | 2 +- .../add-forge-now-request-dashboard.spec.js | 7 +- .../migrations/0005_prepare_inbound_email.py | 30 ++++++++ swh/web/add_forge_now/models.py | 26 +++++++ swh/web/add_forge_now/views.py | 32 +++++++++ swh/web/api/views/add_forge_now.py | 20 +++++- swh/web/config.py | 1 + swh/web/tests/add_forge_now/test_models.py | 14 +++- swh/web/tests/api/views/test_add_forge_now.py | 83 +++++++++++++++++++++- 10 files changed, 209 insertions(+), 9 deletions(-) create mode 100644 swh/web/add_forge_now/migrations/0005_prepare_inbound_email.py
Changes applied before test
commit 3f4da941a759444ade8860c060769629cd12a829
Author: Nicolas Dandrimont <nicolas@dandrimont.eu>
Date: Fri May 6 18:36:48 2022 +0200
Add forge now: add view to retrieve RequestHistory.raw_message
To access this view, one can follow the link inside RequestHistory API
responses.
commit 3af94b02c2ee2bd92be18650d4ae80c61465083f
Author: Nicolas Dandrimont <nicolas@dandrimont.eu>
Date: Fri Apr 1 17:08:47 2022 +0200
Add forge now: prepare fields for inbound email processing
- add raw message field to request history table
- add email as choice for the request history actor roles, when the
message can't be matched to a known actor
commit 9b80cf37065930c6cee71a5558a349e48a12e2da
Author: Nicolas Dandrimont <nicolas@dandrimont.eu>
Date: Tue May 10 12:35:51 2022 +0200
Add Forge Now: add a forge_domain property for use in templates
This lets us generate more meaningful email subjects for request
tracking, and will be useful for raw message filename generation too.
commit 833b171c69325d863fb06d0ec603ebd0be9f8b6c
Author: Nicolas Dandrimont <nicolas@dandrimont.eu>
Date: Fri Apr 1 10:38:38 2022 +0200
Add forge now: add inbound email address to modelSee https://jenkins.softwareheritage.org/job/DWAPPS/job/tests-on-diff/1816/ for more details.
Comment Actions
Yeah, that's a good idea.
I'll bundle that up in the next rebase. Any other suggestions?
Comment Actions
Build is green
Patch application report for D7799 (id=28191)
Rebasing onto 3fc9c9276c...
Current branch diff-target is up to date.
Changes applied before test
commit 5d37baf27ec046ed7c54ce92ebc13f8cb753d51d
Author: Nicolas Dandrimont <nicolas@dandrimont.eu>
Date: Fri May 6 18:36:48 2022 +0200
Add forge now: add view to retrieve RequestHistory.message_source
To access this view, one can follow the link inside RequestHistory API
responses.
commit 633b4319c4ad0e3dbefe13552d7af86c449b50f7
Author: Nicolas Dandrimont <nicolas@dandrimont.eu>
Date: Fri Apr 1 17:08:47 2022 +0200
Add forge now: prepare fields for inbound email processing
- add message source field to request history table
- add email as choice for the request history actor roles, when the
message can't be matched to a known actorSee https://jenkins.softwareheritage.org/job/DWAPPS/job/tests-on-diff/1821/ for more details.