Skip to content

Conversation

apskhem
Copy link
Collaborator

@apskhem apskhem commented Aug 20, 2025

Description

  • Removed POST /v1/document endpoint entirely
  • Removed related integration test to the endpoint
  • Removed Full signed document query in the event-db as it's no longer used

Related Issue(s)

Closes #3199

Please confirm the following checks

  • My code follows the style guidelines of this project
  • 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
  • My changes generate no new warnings
  • 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
  • Any dependent changes have been merged and published in downstream module

@apskhem apskhem self-assigned this Aug 20, 2025
@apskhem apskhem requested a review from kukkok3 as a code owner August 20, 2025 11:56
@apskhem apskhem added the review me PR is ready for review label Aug 20, 2025
@apskhem apskhem added the squad: gatekeepers Catalyst App Backend, System Development & Integration Team label Aug 20, 2025
@apskhem apskhem linked an issue Aug 20, 2025 that may be closed by this pull request
2 tasks
@apskhem apskhem added this to Catalyst Aug 20, 2025
@apskhem apskhem moved this from New to 🏗 In progress in Catalyst Aug 20, 2025
@apskhem apskhem moved this from 🏗 In progress to 👀 In review in Catalyst Aug 20, 2025
@damian-molinski
Copy link
Contributor

Don't merge it until we're all ready for it please

@damian-molinski damian-molinski added fyi: cot For information - Catalyst Ops & Tools Team fyi: face For information - Frontend Catalyst Experience Team labels Aug 20, 2025
@apskhem apskhem added the do not merge yet PR is not ready to be merged yet label Aug 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do not merge yet PR is not ready to be merged yet fyi: cot For information - Catalyst Ops & Tools Team fyi: face For information - Frontend Catalyst Experience Team review me PR is ready for review squad: gatekeepers Catalyst App Backend, System Development & Integration Team
Projects
Status: 👀 In review
Development

Successfully merging this pull request may close these issues.

[Bug]: V1 POST document/index pagination problem
2 participants