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

Tests on the landingpage with XPath and opacity #5080

Draft
wants to merge 2 commits into
base: release_3_8
Choose a base branch
from

Conversation

Gustry
Copy link
Member

@Gustry Gustry commented Dec 4, 2024

I can confirm that reverting PR #5011 solves the 500.

The purpose was not to merge this PR, but to understand why the E2E was OK on PR #5011.
While there is a Cypress test about the landing page :
https://github.com/3liz/lizmap-web-client/blob/release_3_8/tests/end2end/cypress/integration/projects_homepage-ghaction.js

I can confirm there is a cache issue.

I tried to empty the redis database :

➜ docker exec -it lizmaprelease_3_8_test_redis redis-cli flushall
OK

@github-actions github-actions bot added this to the 3.8.4 milestone Dec 4, 2024
@Gustry Gustry changed the title Xpath opacity Tests on the landingpage with XPath and opacity Dec 4, 2024
@Gustry Gustry marked this pull request as draft December 4, 2024 13:35
@Gustry
Copy link
Member Author

Gustry commented Dec 4, 2024

CC @mind84 and @rldhont

I just want to trust our E2E test suite and avoid any regression like this one.

@mind84
Copy link
Contributor

mind84 commented Dec 4, 2024

Hi @Gustry @rldhont,

I'm preparing the fix for the 500 error. Basically consists in a manual backport of the commit 1e9be74

@rldhont
Copy link
Collaborator

rldhont commented Dec 4, 2024

I also working on speed up main view to parse only project main data. #5081

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.

3 participants