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

Deactivate point observation routes when not desired and fix duplicate observations on areaSheet #237

Merged
merged 6 commits into from
Nov 25, 2020

Conversation

lpofredc
Copy link
Contributor

  • Désactive les routes diffusant des données précises lorsque l'option "AFFICHAGE_MAILLE" est activé (sinon restent disponibles par API)
  • Le script de génération des observations points d'espèces par communes dupliquait les données (deux push successifs)

@camillemonchicourt
Copy link
Member

Joli ! :)

@camillemonchicourt
Copy link
Member

On prépare une release corrective. La duplication des observations est corrigé déjà dans la develop : bc19808
Mais j'intégrerai bien l'autre correction apportée dans cette PR.
@TheoLechemia et @jpm-cbna OK pour vous ?

@TheoLechemia
Copy link
Member

Oui Ok pour moi

@jpm-cbna
Copy link
Contributor

C'est bon.
Mais j'aurai plutôt renvoyé une réponse"403 Forbidden" avec un message d'erreur approprié dans la route plutôt que de supprimer la route lors de l'affichage par maille.
Quelque chose dans ce genre:

    # Check if permissions management is enable
    if not current_app.config["PERMISSION_MANAGEMENT"]["ENABLE_ACCESS_REQUEST"]:
        response = {
            "message": "Demande de permissions d'accès non activé sur cette instance de Geonature.",
            "status": "warning"
        }
        return response, 403

@camillemonchicourt camillemonchicourt merged commit 6f3b671 into PnX-SI:develop Nov 25, 2020
@camillemonchicourt
Copy link
Member

OK merci.
@jpm-cbna j'ai mergé dans Develop. Si tu veux améliorer directement dans la branche Develop, sinon on garde ça en amélioration pour plus tard.

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

5 participants