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

Rename Hyrax.collection.browse_view to edit_view & update value #4041

Merged
merged 1 commit into from
Oct 2, 2019

Conversation

no-reply
Copy link
Contributor

This view is more properly called the 'Edit View' and that should be reflected
in the breadcrumbs. The i18n key was previously also browse_view and changing
it to edit_view helps ensure translations get updated.

Fixes #2164.

Changes proposed in this pull request:

  • Change the i18n key for the collection edit view to edit_view
  • Update text to match.

Guidance for testing, such as acceptance criteria or new user interface behaviors:

  • Go to a collection edit page; observe breadcrumb name "Edit View"

@samvera/hyrax-code-reviewers

This view is more properly called the 'Edit View' and that should be reflected
in the breadcrumbs. The i18n key was previously also `browse_view` and changing
it to `edit_view` helps ensure translations get updated.

Fixes #2164.
@no-reply no-reply force-pushed the hyrax-collection-edit-view-key branch from 4b42e1e to 493967a Compare September 24, 2019 01:13
Copy link
Member

@cjcolvar cjcolvar left a comment

Choose a reason for hiding this comment

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

👍

@cjcolvar cjcolvar merged commit 49f4ca4 into master Oct 2, 2019
@cjcolvar cjcolvar deleted the hyrax-collection-edit-view-key branch October 2, 2019 16:28
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.

Breadcrumbs on collections edit view is titled "Browse View"
2 participants