Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

v3-modify-asset #93

Merged
merged 5 commits into from
Sep 18, 2024
Merged

Conversation

quirijnslings
Copy link
Contributor

NOTE: please merge v3-platform first!

New features:

  • Allow changing the datePublished on an existing Media item
  • Make it possible to remove metaproperty values from an asset

Bugfix:

  • Prevent unintentional changing of Archive and IsPublic properties on assets

Unit tests covering these changes:

  • QueryParameterTest

Sample program to test these changes:

  • ModifyMediaSample

@ahongbynder
Copy link
Contributor

@quirijnslings With the previous merges noticing merge conflicts for this PR as well as for find-assets, tags, and upload.

@quirijnslings
Copy link
Contributor Author

@ahongbynder I'm not surprised. I was asked to split the changes into as many PRs as possible, but many changes occur in the same files. I will merge them and push again

@quirijnslings
Copy link
Contributor Author

quirijnslings commented Sep 18, 2024

I resolved the merge conflicts (in the other 3 PRs as well)

@ahongbynder ahongbynder merged commit e472748 into Bynder:master Sep 18, 2024
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