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

#1819 - Epingler une observation #1843

Closed
wants to merge 4 commits into from
Closed

#1819 - Epingler une observation #1843

wants to merge 4 commits into from

Conversation

Gaetanbrl
Copy link
Contributor

@Gaetanbrl Gaetanbrl commented Apr 11, 2022

Ref. issue #1819

Cette contribution a été réalisée selon les besoins de picardie-nature @jbdesbas

###Description

Cette proposition de PR permet, depuis les modules VALIDATION et SYNTHESE, de pouvoir épingler une observation afin de la retrouver facilement depuis le module Validation (dans la liste).

Les règles sont les suivantes :

  • Tous les utilisateurs qui ont les droits suivants peuvent créer ou supprimer une épingle :
/POST reports type pin
@permissions.check_cruved_scope("R", get_scope=True, module_code="SYNTHESE")
/DELETE reports type pin
@permissions.login_required
  • 1 épingle par observation par utilisateur uniquement
  • Seul le propriétaire de l'épingle peut la supprimer ou l'ajouter (ou un admin en base)
  • Un utilisateur ne peut voir que les épingles qu'il a créé lui-même
  • Une épingle et un "report" de la table t_report (idem discussion, signalement)
  • Les épingles peuvent être filtrées dans la recherche avancée (idem signalement)
  • Les épingles sont visible dans le tableau de validation via un icône en forme de punaise (idem signalement)
  • Cette fonctionnalité est optionnelle car visible ou non dans les modules SYNTHESE, VALIDATION ou dans aucun des deux selon les détails fourni dans l'issue Possibilité d'épingler une occurrence #1819.

Preview

pr-pin

@Gaetanbrl Gaetanbrl changed the title Create Pin tool #1819 - Epingler une observation Apr 11, 2022
first pin func

Merge develop and pin oldest work

backend and front linter

simpliest reports update on modal clos

avoid db conflict on geonature db update

linter applied

comment pin code

better aler, pin icons UI

remove validator right only for pin
@codecov
Copy link

codecov bot commented Apr 11, 2022

Codecov Report

Merging #1843 (b72f21a) into develop (99eaab7) will increase coverage by 0.00%.
The diff coverage is 66.66%.

@@           Coverage Diff            @@
##           develop    #1843   +/-   ##
========================================
  Coverage    54.60%   54.61%           
========================================
  Files           76       76           
  Lines         7292     7295    +3     
========================================
+ Hits          3982     3984    +2     
- Misses        3310     3311    +1     
Flag Coverage Δ
pytest 54.61% <66.66%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...nature/core/gn_synthese/utils/query_select_sqla.py 72.72% <50.00%> (-0.28%) ⬇️
backend/geonature/core/gn_synthese/routes.py 78.51% <66.66%> (ø)
backend/geonature/utils/config_schema.py 91.73% <100.00%> (+0.03%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 99eaab7...b72f21a. Read the comment docs.

@Gaetanbrl
Copy link
Contributor Author

merci @bouttier pour la revue

@bouttier
Copy link
Contributor

supersed by #1929

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.

2 participants