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

[dev-branch] Media url mapping tool does not report errors #2117

Open
themerekat opened this issue Jan 24, 2025 · 2 comments
Open

[dev-branch] Media url mapping tool does not report errors #2117

themerekat opened this issue Jan 24, 2025 · 2 comments
Assignees
@MuchQuak
Labels
enhancement

Comments

@themerekat
Copy link
Collaborator

I uploaded a file of images that already exist in the portal and they didn't get attached to their records, I assume because they are duplicates. The url mapping report doesn't give any indication that there were errors, but also doesn't say that the upload was successful.

Could use better error handling

Image
@themerekat themerekat changed the title [dev-branch] Media url mapping file does not report errors [dev-branch] Media url mapping tool does not report errors Jan 24, 2025
@themerekat
Copy link
Collaborator Author

Also there is an unnecessary phrase at the end of the report. This phrase applies to when you do a determinations upload, but not an images upload.

Image

@MuchQuak MuchQuak self-assigned this Jan 28, 2025
MuchQuak added a commit that referenced this issue Jan 29, 2025
@MuchQuak
Closes #2117 Url Mapping Upload Adjustments
c24c012 
# Issue #2117

# Summary
Adds Required Fields for media upload `format` and `originalUrl` must
now be mapped. There is a js alert for when they aren't. Subsequently if
the values are null the row will not be imported and a notification will
be show stating if either values are null
@MuchQuak MuchQuak mentioned this issue Jan 29, 2025
Open
20 tasks
@themerekat
Copy link
Collaborator Author

Image

Image

Image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@MuchQuak MuchQuak

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@themerekat @MuchQuak