-
Notifications
You must be signed in to change notification settings - Fork 148
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
The branding location changes is a breaking change #941
Comments
I would like to say this is not a bug but more of a breaking change. In this PR the code that loads the branding template completely ignores the I'm currently using liquid templates and this pretty much breaks the use for them. |
Created this PR: #942 |
I've updated the issue name to reflect it, you're right |
We've just come across this. Worth reverting the original change and getting a new version out? |
DXCDT-689: Fix branding template body path parsing issue Issue The branding location changes is a breaking change #941
A fix for this has been provided in 7.24.2. Please update and advise if you have any further issue. |
DXCDT-689: Fix branding template body path parsing issue Issue The branding location changes is a breaking change #941
Checklist
Description
The recent changes to the branding template path have caused a bug in the deploy pipeline: 4b24d78:
Expectation
The deploy to work as expected
Reproduction
Just run the deploy to auth0 using the CLI
Deploy CLI version
7.24.1
Node version
20.x
The text was updated successfully, but these errors were encountered: