Skip to content

Conversation

@AndresMorelos
Copy link
Contributor

  • Added a new POST endpoint to retrieve specific destinations by their UUIDs, allowing for batch operations with a maximum of 100 IDs per request.
  • Updated API documentation to reflect the new endpoint and its parameters, including optional authentication for enhanced filtering.
  • Improved the handling of connected users in the destinations logic, ensuring real-time data is reflected in user counts.
  • Refactored related utility functions and models to support the new functionality and maintain code clarity.

…eval

- Added a new POST endpoint to retrieve specific destinations by their UUIDs, allowing for batch operations with a maximum of 100 IDs per request.
- Updated API documentation to reflect the new endpoint and its parameters, including optional authentication for enhanced filtering.
- Improved the handling of connected users in the destinations logic, ensuring real-time data is reflected in user counts.
- Refactored related utility functions and models to support the new functionality and maintain code clarity.
@github-actions
Copy link

Pull Request Test Coverage Report for Build 21291512123

Details

  • 250 of 271 (92.25%) changed or added relevant lines in 4 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.04%) to 91.545%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/entities/Destination/utils.ts 78 83 93.98%
src/entities/Destination/routes/getDestinationsListById.ts 157 173 90.75%
Totals Coverage Status
Change from base Build 21254943655: -0.04%
Covered Lines: 14586
Relevant Lines: 15692

💛 - Coveralls

Copy link
Contributor

@kevinszuchet kevinszuchet left a comment

Choose a reason for hiding this comment

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

LGTM

@AndresMorelos AndresMorelos merged commit 8f982c3 into master Jan 23, 2026
3 checks passed
@AndresMorelos AndresMorelos deleted the feat/re-discovery-post-destinations branch January 23, 2026 16:28
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.

3 participants