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

Add ux configuration to charts. Allow callers to override default UX. #15

Merged
merged 4 commits into from
Jun 16, 2021

Conversation

usingtechnology
Copy link
Contributor

Signed-off-by: Jason Sherman jsherman@parcsystems.ca

Relates to hyperledger-labs/business-partner-agent#477. This supports the configuration changes in that PR.

Jason Sherman added 2 commits June 14, 2021 15:11
Signed-off-by: Jason Sherman <jsherman@parcsystems.ca>
Signed-off-by: Jason Sherman <jsherman@parcsystems.ca>
Copy link
Contributor

@frank-bee frank-bee left a comment

Choose a reason for hiding this comment

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

Great stuff!!
The most of my comments are just optional hints. But the construct near line 275 might miss something.

charts/bpa/templates/_helpers.tpl Outdated Show resolved Hide resolved
charts/bpa/templates/bpa_configmap_ux.yaml Outdated Show resolved Hide resolved
charts/bpa/values.schema.json Outdated Show resolved Hide resolved
charts/bpa/values.yaml Outdated Show resolved Hide resolved
Jason Sherman added 2 commits June 15, 2021 15:26
…default". Remove global.ux from form configuration.

Signed-off-by: Jason Sherman <jsherman@parcsystems.ca>
…default". Remove global.ux from form configuration.

Signed-off-by: Jason Sherman <jsherman@parcsystems.ca>
@frank-bee frank-bee merged commit ee619f7 into hyperledger-labs:main Jun 16, 2021
usingtechnology added a commit that referenced this pull request Oct 12, 2021
Update schemas and security for POC Demo
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants