-
Notifications
You must be signed in to change notification settings - Fork 32
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 bulk actions to POIs and events #1105
Conversation
2ac6f5e
to
d629727
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
3c582b4
to
9a2e8f3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! 🚀
See my comments below for a few minor improvements.
One additional problem I noticed: The bulk action form interferes with the duplication forms for events, I'm afraid we have to find some workaround for this. In theory, I don't think nested forms are allowed in HTML 😒
f5a55c3
to
0ad4aa2
Compare
3f52223
to
fb5079e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! I still don't think the spacing looks consistent yet, so I added a few more suggestions.
d2edf32
to
74789ec
Compare
3e9f513
to
d04a7bd
Compare
Co-authored-by: Timo Ludwig <ludwig@integreat-app.de>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks a lot, I think this is ready to be merged now! 🎉
@ulliholtgrave I think you have to approve as well, or at least somehow delete your "changes requested" status in order for this PR to be merged ;) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks fine now! :)
Short description
Add checkboxes to pois and events, add option to auto translate
Proposed changes
Add the dropdown from posts to events and pois using the existing bulk action template.
Resolved issues
Fixes: #1103
Addition information
This is part of the project for Johannes' final exam.