-
Notifications
You must be signed in to change notification settings - Fork 1
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
features/platform and organisation vocabs search #83
Conversation
Ah wrong one, this one not ready yet |
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.
Minor comment, and please name your branch with either features/ bugs/ testing/ then follow by ticket id and then a description
bugs/2342-fix-buid-issue etc
I approve this ticket and please do not forget to update appDeploy if needed
parameter_vocab: | ||
path: parameter_vocab | ||
field: label | ||
search_suggestions: |
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.
Need change to https://github.com/aodn/appdeploy/blob/main/tg/edge/ogcapi-java/ecs/variables.yaml and other env?
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.
I'm not sure if it will be straight forward to rename a remote branch than already has a reviewed PR, I might just rename the PR name this time.
No description provided.