Skip to content

Setup api endpoint for vigilance areas - #5600

Draft
PartyNell wants to merge 48 commits into
vigilance_areafrom
vigilance_area_api_v2
Draft

Setup api endpoint for vigilance areas#5600
PartyNell wants to merge 48 commits into
vigilance_areafrom
vigilance_area_api_v2

Conversation

@PartyNell

Copy link
Copy Markdown
Contributor

Description

Add endpoints for vigilance area module and add filters on treks endpoint to manipulate opened/closed treks.

Related Issue

Checklist

  • I have followed the guidelines in our Contributing document
  • My code respects the Definition of done available in the Development section of the documentation
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works.
  • New and existing unit tests pass locally with my changes
  • I added an entry in the changelog file
  • My commits are all using prefix convention (emoji + tag name) and references associated issues
  • I added a label to the PR corresponding to the perimeter of my contribution
  • The title of my PR mentionned the issue associated

AI requirements

Skip the checkboxes below 👇 If you didn't use AI for your contribution

  • I used AI assistance to produce part or all of this contribution
  • I have read, reviewed, understood and can explain the code I am submitting
  • I can explain and discuss my work to a maintainer
  • I pasted the prompt used and mentioned the model used below

@PartyNell
PartyNell changed the base branch from master to vigilance_area July 29, 2026 16:13
@cypress

cypress Bot commented Jul 29, 2026

Copy link
Copy Markdown

Geotrek-admin    Run #17613

Run Properties:  status check passed Passed #17613  •  git commit f452ddc315 ℹ️: Merge c36fff818175f36639b1aa0171f6be2f850a3f48 into 3f5d1a016fdbe875c30188d8496a...
Project Geotrek-admin
Branch Review refs/pull/5600/merge
Run status status check passed Passed #17613
Run duration 01m 48s
Commit git commit f452ddc315 ℹ️: Merge c36fff818175f36639b1aa0171f6be2f850a3f48 into 3f5d1a016fdbe875c30188d8496a...
Committer Nell Party
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 22
View all changes introduced in this branch ↗︎

@submarcos submarcos left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pas mal !

Attention, dans ma PR j'a i"squash" les migrations. hesite pas à détruire ta BDD / recreer et rejouer les migrations

msgstr "Filtrer par un ou plusieurs id de portails, séparés par des virgules."

#, fuzzy
#| msgid "Practices"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

attention à bien verifier les messages fuzzy (et enlever les ocmmentaires). ce sont des trads suggérées par django, souvent à coté de la plaque

@PartyNell
PartyNell force-pushed the vigilance_area_api_v2 branch 3 times, most recently from 397a040 to a2a5f62 Compare August 4, 2026 08:26
@codecov

codecov Bot commented Aug 4, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.56%. Comparing base (da01e2b) to head (c36fff8).

Additional details and impacted files
@@               Coverage Diff                @@
##           vigilance_area    #5600    +/-   ##
================================================
  Coverage           98.55%   98.56%            
================================================
  Files                 277      278     +1     
  Lines               23321    23508   +187     
================================================
+ Hits                22985    23171   +186     
- Misses                336      337     +1     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@PartyNell
PartyNell force-pushed the vigilance_area_api_v2 branch from 176a31b to 57ff447 Compare August 10, 2026 10:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants