Page MenuHomeSoftware Heritage

Integrate swh badges in Permalinks tab
ClosedPublic

Authored by anlambert on Nov 21 2019, 12:17 PM.

Details

Summary

Add swh badges for each type of browsed objects in the Permalinks tab
accessible from the handle on the right of the web UI.

When clicking on one of those badge, a modal detailing how to integrate
it in a README file will be displayed.

Related to T2092

Depends on D2310

Test Plan

Cypress tests have been added.

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

Update: Remove extra </a> in template file

vlorentz added inline comments.
cypress/integration/persistent-identifiers.spec.js
229–243

Are these .wait(500) needed? cy.get should already waits as long as needed

253

Why is this one needed?

cypress/integration/persistent-identifiers.spec.js
253

Because there is some fade in / fade out animations with bootstrap modals when displaying / closing them that mess up with cypress. Tests were failing without these wait instructions.

I messed up somewhere when updating that diff and now arc patch fails on Jenkins.

Calling arc patch --diff 8021 --nocommit --nobranch --force locally on my master branch works so I don't know what's wrong here.

The problem is: "Base commit is not in local repository; trying to fetch.". You should rebase on a commit in master.

This revision is now accepted and ready to land.Nov 22 2019, 4:54 PM

I messed up somewhere when updating that diff and now arc patch fails on Jenkins.
Calling arc patch --diff 8021 --nocommit --nobranch --force locally on my master branch works so I don't know what's wrong here.

The problem is: "Base commit is not in local repository; trying to fetch.". You should rebase on a commit in master.

So arc patch previously failed for that diff because of the following: D2320 depends on D2310, itself depending on D2309.
But D2309 has landed on master since then and as it was still declared in the dependency chain of D2320, arc patch will try to
unconditionally cherry-pick the associated commit to the current master branch which will fail as the commit already exists.

There is a related task opened on Phabricator forge but no action have been taken so far.

Hopefully, there is a mean to update a diff dependency chain from the Phabricator Web UI (removing Depends on DXXX from the task
description is not enough).

In our case we want to remove D2309 as parent diff of D2310. In the Web UI for D2310, click on Edit Related Revisions
then Edit Parent Revisions. We can then remove D2309 as parent diff and save the changes. Rebuilding the diff on Jenkins
will now successfully apply arc patch \o/.