-
Notifications
You must be signed in to change notification settings - Fork 2
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
Update context.js
and constants.js
#1
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Thanks @JSAssassin!
Re syncing js and json issues: I think the theory of this newer template style is that developers should just edit the js and use the |
@davidlehn - good question. Yes, the generated JSON is supposed to be committed to Git. The reason for that is -- for many of the contexts this template serves, the w3id URL resolves to that very same generated JSON file, hosted via Github Pages. |
@dmitrizagidulin Ah yes, I forgot that use case. Would be good to figure out how to write a github action that checks to make sure js and json are in sync. As was seen here, that can be an issue and is difficult to spot. |
Added tracking issue: digitalbazaar/context-template#6 |
No description provided.