-
Notifications
You must be signed in to change notification settings - Fork 496
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
Dataverse URL: Pretty-fied URL Has Typo #1142
Comments
I think this is all set, no? |
OK, typo and view is gone. Mike pointed out the dataverse url on the dataset card is not prettified so passing to Mike for cleanup. |
|
…e ID's, which don't work with the PrettyFaces URLs. [ref #1142]
Fixed dataverse URLs on search-include that still referenced dataverse ID's, which don't work with the PrettyFaces URLs. Found that this bug in the search-include (as well as the "View All Datasets and Dataverses >" link hidden under the "Preview Recently Published Datasets [+]" link) were easy fixes, so I checked those in quick. Please forgive the dup comment above. |
Fixed, closing. |
When I was working on the ticket #1111 to apply the "pretty" URL to the Featured Dataverse links, I noticed that the "pretty" URL that is being used on the dataverse cards, points to "/view-datataverse/{alias}". There is an extra "T-A" in there. Also, see here on dvn-build.
https://dvn-build.hmdc.harvard.edu/view-datataverse/umaTop
Since we're going to need to go ahead and correct this, I would like to take this opportunity to remove "view-" from the URL as well, since it shouldn't be needed, and "dataverse" is shorter, and sweeter.
The text was updated successfully, but these errors were encountered: