-
Notifications
You must be signed in to change notification settings - Fork 72
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
Auto-Rendering Profile #806
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.
The profile created from GitHub actions looks fine (after updating hierarchy and redirections which do not work well in GitHub actions). However, some information is missing and should be added to this PR.
@sneumann @gtsueng @ivanmicetic could you please add the use cases? They should go to /useCases/DefinedTerm and also the group this profile should go to? I would suggest Dataset.
There are no embedded examples in the page, cannot those be created with DDE as well? If not, see if you can add a couple by hand. Not mandatory/blocking but nice to have.
Thanks.
@sneumann @gtsueng @ivanmicetic any updates? Thanks |
As I can't edit this PR, I have created #808 to replace this one. |
When trying your repo, I get an error for the DefineTerm.html "mapping values are not allowed in this context at line 56 column 12". I will implement your changes here. |
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.
It can go now
Hi, I am still unable to test my changes locally, so thanks for catching |
This pull request has been created by a guithub action, after adding/updating a profile in the specifications repo.