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 : garde la sélection des parcelles lors de l'activation des parcelles environnantes #129

Merged
merged 1 commit into from Apr 17, 2023

Conversation

jillro
Copy link
Member

@jillro jillro commented Apr 14, 2023

No description provided.

Signed-off-by: Maud Royer <hello@maudroyer.fr>
@jillro jillro temporarily deployed to staging April 14, 2023 16:17 — with GitHub Actions Inactive
@netlify
Copy link

netlify bot commented Apr 14, 2023

Deploy Preview for cartobio-dev ready!

Name Link
🔨 Latest commit c88b3c0
🔍 Latest deploy log https://app.netlify.com/sites/cartobio-dev/deploys/64397c9aab7ace0008e8c698
😎 Deploy Preview https://deploy-preview-129--cartobio-dev.netlify.app/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@jillro
Copy link
Member Author

jillro commented Apr 14, 2023

@thom4parisot J'ai beaucoup cherché des solutions beaucoup plus compliquée avant d'arriver à cette solution simple. Je me demande si à terme on ne devrait pas rapproche la logique des layers du container (en terme de composant), puisqu'ils sont inter-dépendants (mais ça rejoint la question de la déduplication du code exploitant / oc (#128).

@thom4parisot
Copy link
Member

Oui tu as raison. Le mix des approches "styles" et API JS de MapLibre ne fait pas bon ménage :/

@thom4parisot thom4parisot merged commit 1acd7d1 into v2 Apr 17, 2023
6 checks passed
@thom4parisot thom4parisot deleted the maud/retours_environs_operateur branch April 17, 2023 09:12
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.

None yet

2 participants