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

I19 add campus features #34

Merged
merged 5 commits into from
Aug 26, 2024
Merged

I19 add campus features #34

merged 5 commits into from
Aug 26, 2024

Conversation

kirkkwang
Copy link

@kirkkwang kirkkwang commented Aug 23, 2024

Story

⚙️ Add campus to ead2_config

619f9af

This commit will add the campus logic to the ead2_config and also change
the rake's reference to the app's version.

🎁 Add campus to breadcrumbs

72b5f47

This commit will add relevant SVG files for the breadcrumbs and a few
helpers as well as some SolrDocument methods that we'll need. The main
thing is adding the Ngao::Arclight::BreadcrumbsHierarchyComponent
because we want to override the breadcrumbs_hierarchy_component view.
Overriding view component views are a little different that regular view
files where if you match the path you'll be set. This required us to
create the component object in this application and then it would be
able to pick up the overridden view component erb.

Ref:

⚙️ Adding a commentted assets debug true

6a1c20d

This is just for the convenience of uncommenting and restarting instead
of needing to type it in every time.

🎁 Add campus to facets

9a17235

This will render the campus in the facets field. All of the changes
came from the current ngao app.

Ref:

🎁 Add campuses to index page

096fc16

This commit will add the campuses to the index page and do some minimum
styling. Also the scroll to top button was added as well all its
accompanying files. The repositories.yml was updated to NGAO's current
one so we can see more than one campus. We also introduced a decorator
pattern to override the repositories controller to add the campus logic.

Expected Behavior Before Changes

Campus was not index and didn't show up on the breadcrumbs. Campus was not a faceted term.

Expected Behavior After Changes

Campus gets indexed and shows up on the breadcrumbs. Campus is now also a faceted term.

Screenshots / Video

image image

image

This commit will add the campus logic to the ead2_config and also change
the rake's reference to the app's version.
This commit will add relevant SVG files for the breadcrumbs and a few
helpers as well as some SolrDocument methods that we'll need.  The main
thing is adding the Ngao::Arclight::BreadcrumbsHierarchyComponent
because we want to override the breadcrumbs_hierarchy_component view.
Overriding view component views are a little different that regular view
files where if you match the path you'll be set.  This required us to
create the component object in this application and then it would be
able to pick up the overridden view component erb.

Ref:
- ViewComponent/view_component#411
This is just for the convenience of uncommenting and restarting instead
of needing to type it in every time.
This will render the campus in the facets field.  All of the changes
came from the current ngao app.

Ref:
- #19
This commit will add the campuses to the index page and do some minimum
styling.  Also the scroll to top button was added as well all its
accompanying files.  The repositories.yml was updated to NGAO's current
one so we can see more than one campus.  We also introduced a decorator
pattern to override the repositories controller to add the campus logic.
Copy link

@sjproctor sjproctor 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 to me!

@kirkkwang kirkkwang merged commit 0e5672b into main Aug 26, 2024
2 checks passed
@kirkkwang kirkkwang deleted the i19-add-campus-features branch August 26, 2024 17:43
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.

2 participants