Skip to content

fix: regression in security.initialUsers.users rendering [#243] - #244

Open
jperville wants to merge 1 commit into
Ontotext-AD:mainfrom
jperville:fix/initial-users-provisioning-regression
Open

jperville wants to merge 1 commit into
Ontotext-AD:mainfrom
jperville:fix/initial-users-provisioning-regression

Conversation

@jperville

Copy link
Copy Markdown

Description

Fix regression in the rendering of security.initialUser.users in users.js

Related Issues

Issue #243

Changes

Add a comma back into the helper so that the generated users.js is valid json again.

Checklist

  • I have tested these changes thoroughly.
  • My code follows the project's coding style.
  • I have added appropriate comments to my code, especially in complex areas.

@jperville

Copy link
Copy Markdown
Author

I tested the fix successfully on my local minikube cluster.

@mihailradkov

Copy link
Copy Markdown
Contributor

I'm currently moving a fix as well in #245 with updated changelog and chart.yaml for a patch release.

This branch has not been deployed

No deployments
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