Skip to content
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

Added default option to AWS secrets manager #633

Merged
merged 6 commits into from
Sep 18, 2023

Conversation

carsongee
Copy link
Contributor

Describe your changes

The provider for AWS is missing the Default option from the UI:
image

Ulan Bekishov recommended I give it a shot to add it myself

Comment Triggers

PR Check triggers
  • Build: trigger build
  • Sub Category Field Check: trigger subcategoryfieldcheck

@carsongee carsongee force-pushed the carsongee/add_default_secret branch from dfbb0a8 to b786103 Compare July 25, 2023 21:53
tiwaritejas
tiwaritejas previously approved these changes Jul 26, 2023
@VikasMaddukuriHarness
Copy link
Contributor

trigger build, subcategoryfieldcheck

Copy link
Collaborator

@rathodmeetsatish rathodmeetsatish left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@VikasMaddukuriHarness @tiwaritejas there are changes in schema and examples so that will require doc change, also add changelog file and add tests to test new changes

@carsongee
Copy link
Contributor Author

I have added to the existing test cases as well as an example to cover this new parameter. I'm happy to add additional test cases if this isn't sufficient. Additionally, I can try and update the auto-docs and schemas. I tried doing it locally, but it produced so many unrelated changes I decided it was better not to push them to be on the safe side. If you have guides or docs somewhere to update those, or you can give me instructions I'm happy to help move this forward. Thanks!

@VikasMaddukuriHarness VikasMaddukuriHarness dismissed stale reviews from tiwaritejas and themself via 0f8b01b August 7, 2023 04:58
@rathodmeetsatish
Copy link
Collaborator

trigger build, subcategoryfieldcheck

@rathodmeetsatish
Copy link
Collaborator

trigger build, subcategoryfieldcheck

@rathodmeetsatish rathodmeetsatish merged commit 5529415 into harness:main Sep 18, 2023
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants