Skip to content
This repository has been archived by the owner on Feb 1, 2023. It is now read-only.

Remove hard coded resource names #30

Closed
d47853 opened this issue Jun 19, 2020 · 0 comments · Fixed by #32
Closed

Remove hard coded resource names #30

d47853 opened this issue Jun 19, 2020 · 0 comments · Fixed by #32
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@d47853
Copy link
Member

d47853 commented Jun 19, 2020

To encourage collaboration some of the hard coded names should be removed from the deployment. These include:

  • The name of the cluster (currently this is a variable in cdk.json but it can be autogenerated)
  • The name of the REST API. This uses the id of the construct so if the ID is set to something like this.node.uniqueId + "API" would work
@d47853 d47853 added enhancement New feature or request good first issue Good for newcomers labels Jun 19, 2020
@d47853 d47853 mentioned this issue Jun 22, 2020
Merged
@d47853 d47853 linked a pull request Jun 22, 2020 that will close this issue
Gh 17 add crud backend #32
Merged
macenturalxl1 added a commit to macenturalxl1/Kai that referenced this issue Jan 12, 2021
@macenturalxl1
Merge pull request gchq#30 from jpelbertrios/gchqgh-7-dockerise-ui
5e473a6 
Gh 07 dockerise ui
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Gh 17 add crud backend gchq/Kai
1 participant
@d47853