Page MenuHomeSoftware Heritage

add_forge_now: Deduplicate request status transitions in js code
ClosedPublic

Authored by anlambert on Aug 18 2022, 4:00 PM.

Details

Summary

Inject transitions dict into Django template instead and pass it to js.

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

Build is green

Patch application report for D8256 (id=29814)

Rebasing onto 568eb5d7ac...

Current branch diff-target is up to date.
Changes applied before test
commit 9027a2823d771010d91920555b11f42b9fb85d81
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date:   Thu Aug 18 15:58:38 2022 +0200

    add_forge_now: Do not duplicate request status transitions in js code
    
    Inject transitions dict into Django template instead and pass it to js.

See https://jenkins.softwareheritage.org/job/DWAPPS/job/tests-on-diff/1977/ for more details.

vlorentz added a subscriber: vlorentz.

s/Do not duplicate/Deduplicate/

swh/web/add_forge_now/models.py
65–70

I'd rather have this in swh/web/admin/add_forge_now.py given that it's only needed for formatting, but ok

This revision is now accepted and ready to land.Aug 18 2022, 4:18 PM

Build is green

Patch application report for D8256 (id=29819)

Rebasing onto 568eb5d7ac...

Current branch diff-target is up to date.
Changes applied before test
commit d72784ac651499ce9875e1288395792efb758e25
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date:   Thu Aug 18 15:58:38 2022 +0200

    add_forge_now: Deduplicate request status transitions in js code
    
    Inject transitions dict into Django template instead and pass it to js.

See https://jenkins.softwareheritage.org/job/DWAPPS/job/tests-on-diff/1978/ for more details.

anlambert retitled this revision from add_forge_now: Do not duplicate request status transitions in js code to add_forge_now: Deduplicate request status transitions in js code.Aug 18 2022, 4:41 PM