-
Notifications
You must be signed in to change notification settings - Fork 3
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 URLs in views #718
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@bsp3ars I made a couple suggestions to your changes. Also the following parts of the issue are unresolved. Let me know if we need to discuss them because replacements don't exit.
- app/views/layouts/scholar/_links.html.erb:http://libraries.uc.edu/digital-scholarship/data-services
- app/views/hyrax/static/agreement.html.erb:http://www.uc.edu/content/dam/uc/provost/docs/academicpersonnel/a-z/AAUP-UC
- Both app/views/welcome_mailer/welcome_email_student.text.erb and app/views/welcome_mailer/welcome_email_student.html.erb have lots of hard-coded links to Scholar pages instead of using URL helpers.
97b3894
to
c372227
Compare
@bsp3ars There's till unresolved stuff here. See #718 (comment) and #718 (review) |
c372227
to
6d3f602
Compare
@hortongn this is ready for a review |
@@ -12,7 +13,7 @@ | |||
<li> | |||
Log in to <%=t('hyrax.product_name') %> using the <a href="#" style="color:white;background-color:black;padding:3px;"> | |||
<span class="glyphicon glyphicon-log-in" aria-hidden="true"></span > Login | |||
</a> link (upper right corner of <a href="http://curly.libraries.uc.edu:3002/">curly.libraries.uc.edu</a>). | |||
</a> link (upper right corner of <%= link_to root_path %>). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should actually be Rails.application.config.application_root_url
instead of root_path
. We want the web page to display "http://some.url" but root_path
just displays "/"
6d3f602
to
ec75989
Compare
@hortongn This should be good to go |
Fixes #699
This should fix all of the listed URLs listed in #699.