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.

Sign up for GitHub

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

Jump to bottom

fix(app-file-manager): append data to not defined field values #3777

Merged
merged 2 commits into from
Jan 4, 2024
Merged

fix(app-file-manager): append data to not defined field values #3777

merged 2 commits into from
Jan 4, 2024

Conversation

leopuleo
Copy link
Contributor

Changes

With this PR we fix a bug found when trying to APPEND new values to a field without data via the File Manager -> Bulk Edit.

How Has This Been Tested?

Jest

Documentation

How to replicate the bug:

  • Select one or more files from the File Manager list: the files should not have previously saved data for the field we are trying to update
  • Click on the "Edit" action
  • Select the field -> "Append to existing values" -> Add some values
  • Submit the operation
CleanShot.2023-12-20.at.17.56.18.mp4

@leopuleo leopuleo added this to the 5.38.3 milestone Dec 20, 2023
@leopuleo leopuleo self-assigned this Dec 20, 2023
@leopuleo leopuleo merged commit fe52405 into dev Jan 4, 2024
6 checks passed
@leopuleo leopuleo deleted the leo/fix/fm-bulk-edit-append branch January 5, 2024 08:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Pavel910 Pavel910 Pavel910 approved these changes

Assignees

@leopuleo leopuleo

Labels
file-manager
Projects
None yet
Milestone
5.38.3
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@leopuleo @Pavel910