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

show breadcrumbs for direct child of root datasets #7257 #8078

Merged
merged 1 commit into from Aug 25, 2021

Conversation

pdurbin
Copy link
Member

@pdurbin pdurbin commented Aug 24, 2021

What this PR does / why we need it:

In Dataverse 4, we always showed breadcrumbs for datasets in root. As part of the Dataverse 5 redesign (in pull request #6909 for #6684), we hid breadcrumbs for datasets in root. This pull request reverts that commit (95d7e04), restoring the behavior back to how it was.

Which issue(s) this PR closes:

Closes #7257

Special notes for your reviewer:

The commit above that changed the behavior between 4 and 5 is mentioned above.

Suggestions on how to test this:

Create datasets in root and non-root collections. Observe that the breadcrumbs are present in both cases.

Does this PR introduce a user interface change? If mockups are available, please link/include them here:

Yes, here's a dataset in the root collection with breadcrumbs.

Screen Shot 2021-08-24 at 2 35 47 PM

Is there a release notes update needed for this change?:

Yes, included.

Additional documentation:

None.

This reverts 95d7e04 which was part of pull request #6909 for #6684.
Copy link
Contributor

@sekmiller sekmiller left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

IQSS/dataverse (TO BE RETIRED / DELETED in favor of project 34) automation moved this from Review 🦁 to QA 🔎✅ Aug 25, 2021
@kcondon kcondon self-assigned this Aug 25, 2021
@kcondon kcondon merged commit 4deb209 into develop Aug 25, 2021
IQSS/dataverse (TO BE RETIRED / DELETED in favor of project 34) automation moved this from QA 🔎✅ to Done 🚀 Aug 25, 2021
@kcondon kcondon deleted the 7257-breadcrumbs branch August 25, 2021 16:19
@djbrooke djbrooke added this to the 5.7 milestone Aug 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

Datasets deposited to root do not have breadcrumb navigation
4 participants