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

Initialize navbar and defaultCatalog for static sites #2467

Open
jrchudy opened this issue May 21, 2024 · 0 comments
Open

Initialize navbar and defaultCatalog for static sites #2467

jrchudy opened this issue May 21, 2024 · 0 comments
Labels
discussion required requires a discussion before moving forward navbar Anything related to the navbar app

Comments

@jrchudy
Copy link
Member

jrchudy commented May 21, 2024

For static sites to use the chaise navbar, currently a chaise-config.js needs to be defined with all of the navbar related configuration properties OR a chaise-config.js with a defaultCatalog to fetch a chaise-config annotation from the catalog object.

When chaise-config.js is not defined, or the value for defaultCatalog is not defined properly, the navbar appears to be broken on static pages. This isn't an issue for chaise apps since chaise requires the catalog information to be present in the page url.

Some ways we can try to resolve this defaultCatalog when it is not defined or chaise-config.js is not defined:

  • query the ermrest:registry in catalog 0 (can be queried as anonymous on dev.derivacloud
    • is it generally a good idea to let anonymous users query this table?
    • pick the first created catalog?
    • maybe add a column to ermrest:registry that signifies the "default catalog"?
  • pick a default catalog value to use in chaise
    • should we use catalog "1" as the default value for defaultCatalog?
    • or using catalog "0" and suggest having a chaise-config annotation defined for that catalog when setting up the server?
@jrchudy jrchudy added discussion required requires a discussion before moving forward navbar Anything related to the navbar app labels Aug 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discussion required requires a discussion before moving forward navbar Anything related to the navbar app
Projects
None yet
Development

No branches or pull requests

1 participant