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

Fix broken links under "Best Practices" (nav bar) in guides, sync up with dataverse.org #8486

Merged
merged 6 commits into from Mar 16, 2022

Conversation

pdurbin
Copy link
Member

@pdurbin pdurbin commented Mar 15, 2022

What this PR does / why we need it:

All the links under "Best Practices" are broken:

Screen Shot 2022-03-15 at 10 04 36 AM

Which issue(s) this PR closes:

Closes IQSS/dataverse.harvard.edu#24

Special notes for your reviewer:

The impression I get from #9216 is that we're trying to avoid Harvard specific content on dataverse.org so I removed the link to "Harvard Dataverse Policies". To add it back in we can just revert 11d943e

Suggestions on how to test this:

Make sure the links work.

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

No.

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

No.

Additional documentation:

None.

@jggautier
Copy link
Contributor

I think @TaniaSchlatter and Dwayne are working on this. I'd defer to them.

@jggautier jggautier removed their request for review March 15, 2022 14:44
@pdurbin pdurbin changed the title Fix broken links under "Best Practices" (nav bar) in guides Fix broken links under "Best Practices" (nav bar) in guides, sync up with dataverse.org Mar 15, 2022
@pdurbin
Copy link
Member Author

pdurbin commented Mar 15, 2022

@TaniaSchlatter and I met to discuss the pull request and decided to sync up the "Best Practices" dropdown in the guides with the one on dataverse.org, which I did in 54228c1.

That is, dataverse.org has these 5 links:

Screen Shot 2022-03-15 at 2 22 40 PM

And now as of this pull request, the guides will have the same 5 links:

Screen Shot 2022-03-15 at 2 24 52 PM

IQSS/dataverse (TO BE RETIRED / DELETED in favor of project 34) automation moved this from Review 🦁 to QA 🔎✅ Mar 15, 2022
@kcondon kcondon merged commit 5b2820f into develop Mar 16, 2022
IQSS/dataverse (TO BE RETIRED / DELETED in favor of project 34) automation moved this from QA 🔎✅ to Done 🚀 Mar 16, 2022
@kcondon kcondon deleted the dhe24 branch March 16, 2022 20:45
@kcondon kcondon self-assigned this Mar 16, 2022
@pdurbin pdurbin added this to the 5.10 milestone Mar 17, 2022
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
4 participants