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

Modified landing page for semiconductors #326

Merged
merged 5 commits into from
Jul 18, 2024

Conversation

chuanlin2018
Copy link
Collaborator

In this check in, filters component has been restructured. A Collection class was created to NIST, Forensics and Semiconductors collection. A new component - Taxonomy was created to display a given collection's taxonomy. Thus we can loop through the collection objects and display the taxonomy.

Currently only two taxonomy data are displayed for each collection. For example, Forensics only display standard NIST taxonomy and Forensics taxonomy. Semiconductors only display standard NIST taxonomy and semiconductors taxonomy. With the new structure, it's much easier to add more collections for display if necessary.

I am also trying to make the color scheme configurable for each collection. The filter component is working but the menu bar at the right side still hard coded.

To test it locally, use data.nist.gov as backend:
http://localhost:4200/lps/pdr0-0001 - for Forensics

Need sample data to test semiconductors. I modified Forensics data to be the fake semiconductor data for testing.

Copy link
Contributor

@deoyani deoyani left a comment

Choose a reason for hiding this comment

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

It is all working.

@RayPlante
Copy link
Collaborator

Please do not merge yet.

@RayPlante RayPlante merged commit d2095ab into integration Jul 18, 2024
2 checks passed
@RayPlante RayPlante deleted the feature/semiconductors branch July 18, 2024 21:25
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.

3 participants