Skip to content

Conversation

@rhysrevans3
Copy link
Collaborator

@rhysrevans3 rhysrevans3 commented Jun 17, 2025

Added validation for patch requests:

  • Convert operations to partial Item.
  • Deny move or copy operations. This might need to be made more lenient if we need move for alternate asset updates.
  • Added expected extensions on a per collection basis. With defaults and regexes. (currently through settings but could be a service)
  • Validate against expected extensions. (ignore required errrors)

closes #22

@rhysrevans3 rhysrevans3 mentioned this pull request Jun 17, 2025
Copy link
Collaborator

@sturoscy-personal sturoscy-personal left a comment

Choose a reason for hiding this comment

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

Going ahead and approving this so we can keep moving forward. Will run locally and hopefully get a dev/test instance running in AWS space.

Copy link
Collaborator

@sturoscy-personal sturoscy-personal left a comment

Choose a reason for hiding this comment

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

Thought I had approved this already. Looks good. Thank you!

@rhysrevans3 rhysrevans3 marked this pull request as ready for review June 24, 2025 08:48
Copy link
Collaborator

@sturoscy-personal sturoscy-personal left a comment

Choose a reason for hiding this comment

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

Thanks Rhys for the work. Approving to move work forward.

@sturoscy-personal sturoscy-personal merged commit ff508f2 into main Jun 25, 2025
1 check passed
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.

PATCH

3 participants