Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Display withdrawn work versions in the dashboard #889

Merged
merged 4 commits into from
Mar 1, 2021

Conversation

awead
Copy link
Contributor

@awead awead commented Feb 25, 2021

Adds a status badge and detail messages for withdrawn work versions so that they will appear correctly in the dashboard and the resource page.

Fixes #805

Screen Shot 2021-02-25 at 2 00 47 PM

@awead awead force-pushed the i#805-withdrawn-state branch from ce3eb74 to 7ea4d0c Compare February 25, 2021 19:18
Adds a status badge and detail messages for withdrawn work versions so
that they will appear correctly in the dashboard and the resource page.
@awead awead force-pushed the i#805-withdrawn-state branch from 7ea4d0c to 8177569 Compare February 25, 2021 19:42
The tests for adding editors to collections and works were not using
pre-recorded API responses. This was resulting in intermittent test
failures.
If a work has only one published version, and that version is withdrawn,
the work needs to be removed from the index so it is not discoverable.
However, the withdrawn work version must be updated.
When viewing a work's settings, if its only version is withdrawn, we
should return to the version's resource page and not work's.
@awead awead force-pushed the i#805-withdrawn-state branch from 8177569 to 5acab47 Compare February 26, 2021 19:21
@awead awead merged commit a4e2274 into develop Mar 1, 2021
@awead awead deleted the i#805-withdrawn-state branch March 1, 2021 17:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Setting withdrawn state breaks the dashboard
1 participant