Skip to content

Conversation

polypixeldev
Copy link
Member

@polypixeldev polypixeldev commented Jul 23, 2025

Summary of the problem

The built-in browser modal for submitting data about blocks isn't great, and currently you can't edit the parameters of a block without removing it and adding it again. :(

Describe your changes

This PR adds custom modals to add/edit the data of a block, for now all except DonationGoal. To allow this, creating new announcements using a modal has been removed, as modals cannot be easily layered. Clicking on the create button will just direct to the announcements/new page.

Some refactoring has also been done to the block database structure and Stimulus controllers to make rendering them in different contexts (like a turbo stream) easier and with less code.

Screencast.From.2025-07-23.15-21-23.mp4

@polypixeldev polypixeldev requested review from a team as code owners July 23, 2025 19:22
Copy link
Member

@sampoder sampoder left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is a nice improvement - thank you! one thing I noticed was that the validations code from your other PR (#11038) doesn't seem to have carried over:

Screenshot 2025-07-26 at 12 40 18 PM

@polypixeldev
Copy link
Member Author

this is a nice improvement - thank you! one thing I noticed was that the validations code from your other PR (#11038) doesn't seem to have carried over:

just merged main into this PR and added the validation to display in the block modal - thanks for reminding me!

@polypixeldev polypixeldev requested a review from sampoder July 30, 2025 01:15
Copy link
Member

@sampoder sampoder left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should use the standard menu styles here:

Screenshot 2025-08-27 at 3 55 54 PM

Also, please use sentence case. Otherwise, lgtm. I think in the future it'd be nice to have a transaction picker for the transaction link page.

Copy link
Member

@sampoder sampoder left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thank you!

@sampoder sampoder merged commit b6db525 into main Sep 4, 2025
13 checks passed
@sampoder sampoder deleted the polypixeldev/announcement-block-modals branch September 4, 2025 02:18
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