Page MenuHomeSoftware Heritage

browse/revision: Fix snapshot context preservation in revision log view
ClosedPublic

Authored by anlambert on Aug 31 2022, 4:21 PM.

Details

Summary

If the view was browsed in a snapshot context (with origin_url and
snapshot provided as query parameters), the associated query parameters
were not forwarded to the URLs contained in the view.

Diff Detail

Repository
rDWAPPS Web applications
Branch
revision-log-snapshot-context-fix
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 31232
Build 48856: Phabricator diff pipeline on jenkinsJenkins console · Jenkins
Build 48855: arc lint + arc unit

Event Timeline

Build is green

Patch application report for D8364 (id=30188)

Rebasing onto 13751718e9...

Current branch diff-target is up to date.
Changes applied before test
commit 764d6e6d2817cad5f60713e6d1f493bb926a8c13
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date:   Wed Aug 31 16:13:54 2022 +0200

    browse/revision: Fix snapshot context preservation in revision log view
    
    If the view was browsed in a snapshot context (with origin_url and
    snapshot provided as query parameters), the associated query parameters
    were not forwarded to the URLs contained in the view.

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

Build is green

Patch application report for D8364 (id=30198)

Rebasing onto de37826d14...

Current branch diff-target is up to date.
Changes applied before test
commit 86f2e94eb3ebd9735031e475ef71a3b7bd6c7ee8
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date:   Wed Aug 31 16:13:54 2022 +0200

    browse/revision: Fix snapshot context preservation in revision log view
    
    If the view was browsed in a snapshot context (with origin_url and
    snapshot provided as query parameters), the associated query parameters
    were not forwarded to the URLs contained in the view.

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

This revision is now accepted and ready to land.Aug 31 2022, 5:55 PM