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

Add "Update All Models" functionality to bulk edit models #3430

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

fhp
Copy link
Contributor

@fhp fhp commented Jan 18, 2025

Checklist

🚨 Please review the guidelines for contributing to this repository. 🚨

  • Make sure you are making a pull request against our main branch (left side)
  • Check that that your branch is up to date with our main.
  • Make sure you are requesting to pull a topic/feature/bugfix branch (right side). Don't request your main!
  • Check that the tests and code linter both pass.
  • If you're a new contributor, please sign our contributor license agreement.

Warnings

  • This PR will change existing database contents.
  • This PR introduces a breaking change to existing installations.

Summary

This PR enhances the bulk edit functionality by adding an "Update All Models" feature. The new functionality allows moderators to update all filtered models at once, rather than having to select them individually. This is especially useful when updating tags for large collections, or assigning all models with a certain tag to a creator or collection.

Linked issues

Description of changes

image

  • Added "Update All Models" button to bulk edit interface
  • Added translations for the new button in multiple languages (en, de, es, fr, pl). Translations were AI generated and translated back, looks ok, but I'm no native speaker.
  • Enhanced ModelsController to handle bulk updates for all filtered models
  • Add some tests (AI generated, I'm no Ruby developer, but it looked good to me)

@Floppy
Copy link
Collaborator

Floppy commented Jan 18, 2025

Pretty sure there is a "select all" box at the top, perhaps badly signposted. I'm away from my machine now, will check later on.

@Floppy
Copy link
Collaborator

Floppy commented Jan 18, 2025

Unless, and I haven't looked to see if this is the case, the idea of this is to update everything in every page? Which you can't currently do, for sure.

@fhp
Copy link
Contributor Author

fhp commented Jan 18, 2025

update everything in every page

Yes, that is the point :)

I had some tags with 50-200 models in it, that should become collections. Updating that page for page was quite tedious.

And I wanted to remove the !new tag, but didn't even try.

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