Skip to content

add readme more instructions and script to fill some resources after …#5278

Merged
VladMstv merged 2 commits intodevelopfrom
update-deploy-readme-and-scripts
Mar 13, 2026
Merged

add readme more instructions and script to fill some resources after …#5278
VladMstv merged 2 commits intodevelopfrom
update-deploy-readme-and-scripts

Conversation

@VladMstv
Copy link
Copy Markdown
Collaborator

…applying

@VladMstv VladMstv requested a review from colin-welch November 26, 2025 23:42
Copy link
Copy Markdown
Contributor

@colin-welch colin-welch left a comment

Choose a reason for hiding this comment

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

thanks for adding the additional information to the docs!

projectApiUrl: projectApiUrlFromAuth0Audience(secrets.AUTH0_AUDIENCE),
});

// get organization
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

these two organization lookups could use the name as defined in config instead of hardcoded strings

fs.writeFileSync(envFilePath, JSON.stringify(secrets, null, 2));
console.log(`Updated AUTH0_CLIENT_TESTS (${e2eM2M.clientId}) and AUTH0_SECRET_TESTS in ${envFilePath}`);

// create test environment files for apps with E2E M2M credentials
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

could these could be managed by terraform similar to how we now manage the app env files?

@VladMstv VladMstv merged commit 801ad11 into develop Mar 13, 2026
1 check passed
@VladMstv VladMstv deleted the update-deploy-readme-and-scripts branch March 13, 2026 14:29
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