Page MenuHomeSoftware Heritage

browse: Update snapshot branches list view to display all target types
ClosedPublic

Authored by anlambert on Fri, Nov 4, 2:33 PM.

Details

Summary

Previously, the snapshot branches list view was only displaying branches
targeting revisions but it now exists cases where branches can target
contents or directories.

So update view implementation to display all type of branches and add
visual clues to indicate which object types they are targeting.

Depends on D8807

Related to T4669



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 has FAILED

Patch application report for D8814 (id=31755)

Could not rebase; Attempt merge onto d055e08d8b...

Updating d055e08d..f0f3e689
Fast-forward
 swh/web/browse/snapshot_context.py                 | 136 +++++++---
 swh/web/browse/templates/browse-branches.html      |   9 +-
 .../templates/includes/snapshot-context.html       |   6 +-
 .../browse/templates/includes/top-navigation.html  |   4 +-
 swh/web/tests/browse/test_snapshot_context.py      |  22 +-
 swh/web/tests/browse/views/test_content.py         |   8 +-
 swh/web/tests/browse/views/test_directory.py       |  12 +-
 swh/web/tests/browse/views/test_snapshot.py        | 295 +++++++++++++++++++--
 swh/web/tests/conftest.py                          |   4 +
 swh/web/tests/utils/test_archive.py                |   7 +-
 swh/web/utils/archive.py                           |   6 +
 swh/web/utils/typing.py                            |  10 +-
 12 files changed, 425 insertions(+), 94 deletions(-)
Changes applied before test
commit f0f3e6890a2e081c2f868a8e9a6755cd952ae227
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date:   Fri Nov 4 14:26:10 2022 +0100

    browse: Update snapshot branches list view to display all target types
    
    Previously, the snapshot branches list view was only displaying branches
    targeting revisions but it now exists cases where branches can target
    contents or directories.
    
    So update view implementation to display all type of branches and add
    visual clues to indicate which object types they are targeting.
    
    Related to T4669

commit 8f594ec2b6e2e60d49a53fc9ed72d1f3607d1a10
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date:   Wed Nov 2 15:52:59 2022 +0100

    browse: Handle snapshots whose branches target contents or directories
    
    Previously swh-web was only supporting the browsing of snapshot branches
    targeting revisions but we recently introduced new Software Heritage
    loaders producing snapshots whose branches target contents or directories
    but no revisions.
    
    So add support to browse such snapshots as previously they were flagged as
    empty when trying to display them.
    
    Related to T4669

Link to build: https://jenkins.softwareheritage.org/job/DWAPPS/job/tests-on-diff/2108/
See console output for more information: https://jenkins.softwareheritage.org/job/DWAPPS/job/tests-on-diff/2108/console

Build has FAILED

Patch application report for D8814 (id=31755)

Could not rebase; Attempt merge onto d055e08d8b...

Updating d055e08d..f0f3e689
Fast-forward
 swh/web/browse/snapshot_context.py                 | 136 +++++++---
 swh/web/browse/templates/browse-branches.html      |   9 +-
 .../templates/includes/snapshot-context.html       |   6 +-
 .../browse/templates/includes/top-navigation.html  |   4 +-
 swh/web/tests/browse/test_snapshot_context.py      |  22 +-
 swh/web/tests/browse/views/test_content.py         |   8 +-
 swh/web/tests/browse/views/test_directory.py       |  12 +-
 swh/web/tests/browse/views/test_snapshot.py        | 295 +++++++++++++++++++--
 swh/web/tests/conftest.py                          |   4 +
 swh/web/tests/utils/test_archive.py                |   7 +-
 swh/web/utils/archive.py                           |   6 +
 swh/web/utils/typing.py                            |  10 +-
 12 files changed, 425 insertions(+), 94 deletions(-)
Changes applied before test
commit f0f3e6890a2e081c2f868a8e9a6755cd952ae227
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date:   Fri Nov 4 14:26:10 2022 +0100

    browse: Update snapshot branches list view to display all target types
    
    Previously, the snapshot branches list view was only displaying branches
    targeting revisions but it now exists cases where branches can target
    contents or directories.
    
    So update view implementation to display all type of branches and add
    visual clues to indicate which object types they are targeting.
    
    Related to T4669

commit 8f594ec2b6e2e60d49a53fc9ed72d1f3607d1a10
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date:   Wed Nov 2 15:52:59 2022 +0100

    browse: Handle snapshots whose branches target contents or directories
    
    Previously swh-web was only supporting the browsing of snapshot branches
    targeting revisions but we recently introduced new Software Heritage
    loaders producing snapshots whose branches target contents or directories
    but no revisions.
    
    So add support to browse such snapshots as previously they were flagged as
    empty when trying to display them.
    
    Related to T4669

Link to build: https://jenkins.softwareheritage.org/job/DWAPPS/job/tests-on-diff/2109/
See console output for more information: https://jenkins.softwareheritage.org/job/DWAPPS/job/tests-on-diff/2109/console

Harbormaster returned this revision to the author for changes because remote builds failed.Fri, Nov 4, 2:53 PM
Harbormaster failed remote builds in B32708: Diff 31755!

Build was aborted

Patch application report for D8814 (id=31755)

Could not rebase; Attempt merge onto d055e08d8b...

Updating d055e08d..f0f3e689
Fast-forward
 swh/web/browse/snapshot_context.py                 | 136 +++++++---
 swh/web/browse/templates/browse-branches.html      |   9 +-
 .../templates/includes/snapshot-context.html       |   6 +-
 .../browse/templates/includes/top-navigation.html  |   4 +-
 swh/web/tests/browse/test_snapshot_context.py      |  22 +-
 swh/web/tests/browse/views/test_content.py         |   8 +-
 swh/web/tests/browse/views/test_directory.py       |  12 +-
 swh/web/tests/browse/views/test_snapshot.py        | 295 +++++++++++++++++++--
 swh/web/tests/conftest.py                          |   4 +
 swh/web/tests/utils/test_archive.py                |   7 +-
 swh/web/utils/archive.py                           |   6 +
 swh/web/utils/typing.py                            |  10 +-
 12 files changed, 425 insertions(+), 94 deletions(-)
Changes applied before test
commit f0f3e6890a2e081c2f868a8e9a6755cd952ae227
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date:   Fri Nov 4 14:26:10 2022 +0100

    browse: Update snapshot branches list view to display all target types
    
    Previously, the snapshot branches list view was only displaying branches
    targeting revisions but it now exists cases where branches can target
    contents or directories.
    
    So update view implementation to display all type of branches and add
    visual clues to indicate which object types they are targeting.
    
    Related to T4669

commit 8f594ec2b6e2e60d49a53fc9ed72d1f3607d1a10
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date:   Wed Nov 2 15:52:59 2022 +0100

    browse: Handle snapshots whose branches target contents or directories
    
    Previously swh-web was only supporting the browsing of snapshot branches
    targeting revisions but we recently introduced new Software Heritage
    loaders producing snapshots whose branches target contents or directories
    but no revisions.
    
    So add support to browse such snapshots as previously they were flagged as
    empty when trying to display them.
    
    Related to T4669

Link to build: https://jenkins.softwareheritage.org/job/DWAPPS/job/tests-on-diff/2110/
See console output for more information: https://jenkins.softwareheritage.org/job/DWAPPS/job/tests-on-diff/2110/console

Build was aborted

Patch application report for D8814 (id=31755)

Could not rebase; Attempt merge onto d055e08d8b...

Updating d055e08d..f0f3e689
Fast-forward
 swh/web/browse/snapshot_context.py                 | 136 +++++++---
 swh/web/browse/templates/browse-branches.html      |   9 +-
 .../templates/includes/snapshot-context.html       |   6 +-
 .../browse/templates/includes/top-navigation.html  |   4 +-
 swh/web/tests/browse/test_snapshot_context.py      |  22 +-
 swh/web/tests/browse/views/test_content.py         |   8 +-
 swh/web/tests/browse/views/test_directory.py       |  12 +-
 swh/web/tests/browse/views/test_snapshot.py        | 295 +++++++++++++++++++--
 swh/web/tests/conftest.py                          |   4 +
 swh/web/tests/utils/test_archive.py                |   7 +-
 swh/web/utils/archive.py                           |   6 +
 swh/web/utils/typing.py                            |  10 +-
 12 files changed, 425 insertions(+), 94 deletions(-)
Changes applied before test
commit f0f3e6890a2e081c2f868a8e9a6755cd952ae227
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date:   Fri Nov 4 14:26:10 2022 +0100

    browse: Update snapshot branches list view to display all target types
    
    Previously, the snapshot branches list view was only displaying branches
    targeting revisions but it now exists cases where branches can target
    contents or directories.
    
    So update view implementation to display all type of branches and add
    visual clues to indicate which object types they are targeting.
    
    Related to T4669

commit 8f594ec2b6e2e60d49a53fc9ed72d1f3607d1a10
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date:   Wed Nov 2 15:52:59 2022 +0100

    browse: Handle snapshots whose branches target contents or directories
    
    Previously swh-web was only supporting the browsing of snapshot branches
    targeting revisions but we recently introduced new Software Heritage
    loaders producing snapshots whose branches target contents or directories
    but no revisions.
    
    So add support to browse such snapshots as previously they were flagged as
    empty when trying to display them.
    
    Related to T4669

Link to build: https://jenkins.softwareheritage.org/job/DWAPPS/job/tests-on-diff/2111/
See console output for more information: https://jenkins.softwareheritage.org/job/DWAPPS/job/tests-on-diff/2111/console

Build has FAILED

Patch application report for D8814 (id=31755)

Could not rebase; Attempt merge onto d055e08d8b...

Updating d055e08d..f0f3e689
Fast-forward
 swh/web/browse/snapshot_context.py                 | 136 +++++++---
 swh/web/browse/templates/browse-branches.html      |   9 +-
 .../templates/includes/snapshot-context.html       |   6 +-
 .../browse/templates/includes/top-navigation.html  |   4 +-
 swh/web/tests/browse/test_snapshot_context.py      |  22 +-
 swh/web/tests/browse/views/test_content.py         |   8 +-
 swh/web/tests/browse/views/test_directory.py       |  12 +-
 swh/web/tests/browse/views/test_snapshot.py        | 295 +++++++++++++++++++--
 swh/web/tests/conftest.py                          |   4 +
 swh/web/tests/utils/test_archive.py                |   7 +-
 swh/web/utils/archive.py                           |   6 +
 swh/web/utils/typing.py                            |  10 +-
 12 files changed, 425 insertions(+), 94 deletions(-)
Changes applied before test
commit f0f3e6890a2e081c2f868a8e9a6755cd952ae227
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date:   Fri Nov 4 14:26:10 2022 +0100

    browse: Update snapshot branches list view to display all target types
    
    Previously, the snapshot branches list view was only displaying branches
    targeting revisions but it now exists cases where branches can target
    contents or directories.
    
    So update view implementation to display all type of branches and add
    visual clues to indicate which object types they are targeting.
    
    Related to T4669

commit 8f594ec2b6e2e60d49a53fc9ed72d1f3607d1a10
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date:   Wed Nov 2 15:52:59 2022 +0100

    browse: Handle snapshots whose branches target contents or directories
    
    Previously swh-web was only supporting the browsing of snapshot branches
    targeting revisions but we recently introduced new Software Heritage
    loaders producing snapshots whose branches target contents or directories
    but no revisions.
    
    So add support to browse such snapshots as previously they were flagged as
    empty when trying to display them.
    
    Related to T4669

Link to build: https://jenkins.softwareheritage.org/job/DWAPPS/job/tests-on-diff/2113/
See console output for more information: https://jenkins.softwareheritage.org/job/DWAPPS/job/tests-on-diff/2113/console

Build is green

Patch application report for D8814 (id=31755)

Could not rebase; Attempt merge onto d055e08d8b...

Updating d055e08d..f0f3e689
Fast-forward
 swh/web/browse/snapshot_context.py                 | 136 +++++++---
 swh/web/browse/templates/browse-branches.html      |   9 +-
 .../templates/includes/snapshot-context.html       |   6 +-
 .../browse/templates/includes/top-navigation.html  |   4 +-
 swh/web/tests/browse/test_snapshot_context.py      |  22 +-
 swh/web/tests/browse/views/test_content.py         |   8 +-
 swh/web/tests/browse/views/test_directory.py       |  12 +-
 swh/web/tests/browse/views/test_snapshot.py        | 295 +++++++++++++++++++--
 swh/web/tests/conftest.py                          |   4 +
 swh/web/tests/utils/test_archive.py                |   7 +-
 swh/web/utils/archive.py                           |   6 +
 swh/web/utils/typing.py                            |  10 +-
 12 files changed, 425 insertions(+), 94 deletions(-)
Changes applied before test
commit f0f3e6890a2e081c2f868a8e9a6755cd952ae227
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date:   Fri Nov 4 14:26:10 2022 +0100

    browse: Update snapshot branches list view to display all target types
    
    Previously, the snapshot branches list view was only displaying branches
    targeting revisions but it now exists cases where branches can target
    contents or directories.
    
    So update view implementation to display all type of branches and add
    visual clues to indicate which object types they are targeting.
    
    Related to T4669

commit 8f594ec2b6e2e60d49a53fc9ed72d1f3607d1a10
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date:   Wed Nov 2 15:52:59 2022 +0100

    browse: Handle snapshots whose branches target contents or directories
    
    Previously swh-web was only supporting the browsing of snapshot branches
    targeting revisions but we recently introduced new Software Heritage
    loaders producing snapshots whose branches target contents or directories
    but no revisions.
    
    So add support to browse such snapshots as previously they were flagged as
    empty when trying to display them.
    
    Related to T4669

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

Build is green

Patch application report for D8814 (id=31755)

Could not rebase; Attempt merge onto d055e08d8b...

Updating d055e08d..f0f3e689
Fast-forward
 swh/web/browse/snapshot_context.py                 | 136 +++++++---
 swh/web/browse/templates/browse-branches.html      |   9 +-
 .../templates/includes/snapshot-context.html       |   6 +-
 .../browse/templates/includes/top-navigation.html  |   4 +-
 swh/web/tests/browse/test_snapshot_context.py      |  22 +-
 swh/web/tests/browse/views/test_content.py         |   8 +-
 swh/web/tests/browse/views/test_directory.py       |  12 +-
 swh/web/tests/browse/views/test_snapshot.py        | 295 +++++++++++++++++++--
 swh/web/tests/conftest.py                          |   4 +
 swh/web/tests/utils/test_archive.py                |   7 +-
 swh/web/utils/archive.py                           |   6 +
 swh/web/utils/typing.py                            |  10 +-
 12 files changed, 425 insertions(+), 94 deletions(-)
Changes applied before test
commit f0f3e6890a2e081c2f868a8e9a6755cd952ae227
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date:   Fri Nov 4 14:26:10 2022 +0100

    browse: Update snapshot branches list view to display all target types
    
    Previously, the snapshot branches list view was only displaying branches
    targeting revisions but it now exists cases where branches can target
    contents or directories.
    
    So update view implementation to display all type of branches and add
    visual clues to indicate which object types they are targeting.
    
    Related to T4669

commit 8f594ec2b6e2e60d49a53fc9ed72d1f3607d1a10
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date:   Wed Nov 2 15:52:59 2022 +0100

    browse: Handle snapshots whose branches target contents or directories
    
    Previously swh-web was only supporting the browsing of snapshot branches
    targeting revisions but we recently introduced new Software Heritage
    loaders producing snapshots whose branches target contents or directories
    but no revisions.
    
    So add support to browse such snapshots as previously they were flagged as
    empty when trying to display them.
    
    Related to T4669

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

This revision is now accepted and ready to land.Fri, Nov 4, 5:10 PM