Skip to content

feat(ui): add set credential form + action#211

Merged
s0nea merged 1 commit into
clyso:mainfrom
s0nea:wip/storage-credentials-set-action
May 22, 2026
Merged

feat(ui): add set credential form + action#211
s0nea merged 1 commit into
clyso:mainfrom
s0nea:wip/storage-credentials-set-action

Conversation

@s0nea

@s0nea s0nea commented May 7, 2026

Copy link
Copy Markdown
Contributor

Pull Request

Description

Add S3 form:
Screenshot 2026-05-07 at 16 06 37

Edit S3 form:
Screenshot 2026-05-07 at 16 01 53

Add Swift form:
Screenshot 2026-05-07 at 15 52 32

Edit Swift form;
Screenshot 2026-05-07 at 15 52 45

Related Issue

Link to the GitHub issue (if applicable): #[issue number]

Checklist

Note: By submitting this PR, you agree to license your contributions under the Apache 2.0 License and follow our Code of Conduct.

@s0nea s0nea force-pushed the wip/storage-credentials-set-action branch 3 times, most recently from fbec003 to 6017b0e Compare May 20, 2026 12:34
@s0nea s0nea marked this pull request as ready for review May 20, 2026 12:57
@s0nea s0nea force-pushed the wip/storage-credentials-set-action branch from 6017b0e to 4489673 Compare May 20, 2026 12:58
@s0nea s0nea requested a review from mzymta May 20, 2026 12:59
Comment thread ui/src/pages/ChorusSetCredentialPage/ChorusSetCredentialPage.vue Outdated
Comment thread ui/src/components/chorus/set-credential/CredentialFormTile/CredentialFormTile.vue Outdated
@s0nea

s0nea commented May 22, 2026

Copy link
Copy Markdown
Contributor Author

@mzymta thank you for your review. I addressed your comments. Would you mind taking another look at it?

Signed-off-by: Tatjana Dehler <tatjana.dehler@clyso.com>
On-behalf-of: SAP t.dehler@sap.com
@s0nea s0nea force-pushed the wip/storage-credentials-set-action branch from 7ff7c61 to 0defa71 Compare May 22, 2026 07:25
@s0nea s0nea merged commit c6067ea into clyso:main May 22, 2026
1 check passed
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.

2 participants