Skip to content

Conversation

@nkokla
Copy link
Contributor

@nkokla nkokla commented Oct 25, 2025

Mise à jour de BAN-Core-API dans le but d'ajouter un endpoint permettant de demander la publication d'une BAL à partir de son code INSEE. (Seul le endpoint est présent. Son exécution par ID-Fix sera effectuée dans une PR séparée)

@nkokla nkokla marked this pull request as ready for review October 27, 2025 10:45
@nkokla nkokla requested a review from Copilot October 27, 2025 10:45
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds a new /bal/cog endpoint to BAN-Core-API that enables requesting publication of a BAL (Base Adresse Locale) using a COG (INSEE code). The implementation follows the existing pattern of the /bal/text endpoint.

Key Changes:

  • New POST endpoint /bal/cog to handle BAL publication requests by COG
  • Enhanced error messages with more specific context
  • Documentation updates for better clarity and corrections

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.

File Description
apps/ban-core-api/src/index.ts Adds new /bal/cog endpoint with COG-based BAL publication support, improves error messages, and adds type annotations
apps/ban-core-api/index.ts Complete file deletion (likely consolidation or refactoring)
README.md Documentation improvements including typo fixes, formatting cleanup, and new data initialization guide

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@nkokla nkokla marked this pull request as draft October 29, 2025 01:24
nkokla and others added 4 commits October 29, 2025 11:45
…ilot)

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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