Skip to content
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

Fix Display a warning box when the CFG contains warnings #4245

Merged
merged 1 commit into from
Feb 27, 2024

Conversation

rldhont
Copy link
Collaborator

@rldhont rldhont commented Feb 27, 2024

After #4226 and #4242, the warning message can generate the error Uncaught SyntaxError: missing ) after argument list
and Uncaught ReferenceError: lizUrls is not defined. These error messages are generated by a quote in the warning message.

To fix it, we use the Lizmap dictionnary which is a JSON version of the Dictionnary local.

@rldhont rldhont added user interface run end2end If the PR must run end2end tests or not labels Feb 27, 2024
@rldhont rldhont requested review from Gustry and nworr February 27, 2024 10:37
@github-actions github-actions bot added this to the 3.8.0 milestone Feb 27, 2024
After 3liz#4226 and 3liz#4242, the warning message can generate the error `Uncaught SyntaxError: missing ) after argument list`
and `Uncaught ReferenceError: lizUrls is not defined`. These error messages are generated by a quote in the warning message.

To fix it, we use the Lizmap dictionnary which is a JSON version of the Dictionnary local.
@rldhont rldhont force-pushed the fix-project-warnings-display-message branch from 27794d0 to 40b518f Compare February 27, 2024 10:40
@Gustry
Copy link
Member

Gustry commented Feb 27, 2024

unrelated test failure about playwright, it will be fixed in #4246

@Gustry Gustry merged commit 6bd5d79 into 3liz:master Feb 27, 2024
11 of 12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport release_3_7 run end2end If the PR must run end2end tests or not user interface
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants