-
Notifications
You must be signed in to change notification settings - Fork 4
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
SYSENG-1659: add frontier resources #139
Conversation
eca0a49
to
ae952ba
Compare
Depends on #140 and anexia-it/go-anxcloud#311
|
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.
Test CRUD operations on the added frontier resources, works as expected.
Aside from some weird error messages that the engine gives, e.g when a E5E function does not exist, it spits out a 404, which might cause confusion. Maybe we should document that somewhere?
Otherwise LGTM 👍
Forgot to mention it in above comment, could you take a look at the codeclimate issues and fix them? |
014e4f4
to
29afd39
Compare
29afd39
to
04f2884
Compare
looks good to me, successfully tested 👍 |
a7f2f11
to
4aacb0a
Compare
4aacb0a
to
3e137d3
Compare
Description
Added resources to manage the frontier service:
anxcloud_frontier_api
anxcloud_frontier_endpoint
anxcloud_frontier_action
anxcloud_frontier_deployment
Checklist
Unreleased
section in CHANGELOG.md, if user facing changeReferences
Community Note