NASA-PDS / doi-ui

The web interface for the PDS DOI Service providing the ability management PDS archive DOIs. See the DOI Service for more details on the available capabilities. https://nasa-pds.github.io/doi-service/
Apache License 2.0
0 stars 4 forks source link

Update DOI UI app to use NPM package #106

Closed jordanpadams closed 2 years ago

tloubrieu-jpl commented 2 years ago

@eddiesarevalo removes the old search code and will add the npm package version of the search code.

tloubrieu-jpl commented 2 years ago

The change is done and @eddiesarevalo is now testing.

eddiesarevalo commented 2 years ago

@tloubrieu-jpl After some more testing I did discover some bugs. 1) Hovering over status will crash the app. A tooltip is supposed to show but something must be causing an error. 2) Going to the search page will cause the app bar to become white and jumbled. There must be some sort of styling issue.

eddiesarevalo commented 2 years ago

@tloubrieu-jpl Will put this ticket in review and create a new one for bug fixes.

eddiesarevalo commented 2 years ago

Fixed the known bugs will continue to test before closing this one.

tloubrieu-jpl commented 2 years ago

@eddiesarevalo you should have created a pull request in the nasa-pds/doi-ui repository to formalize the update and enable the review of the changes.

tloubrieu-jpl commented 2 years ago

@eddiesarevalo are the changes in pds-wds-install branch ?

gxtchen commented 2 years ago

@jordanpadams @tloubrieu-jpl How can I verify this is done in B12.1?

jordanpadams commented 2 years ago

@gxtchen just updated this to skip I&T