5
11
Fork
You've already forked tomato
1

Add sorting and filtering to site index #163

Merged
edwardloveall merged 2 commits from el-site-glowup into main 2026年06月01日 17:07:21 +02:00

<video src="/attachments/f457443d-8a4f-4980-b91d-84ec2093d629" title="sort and filter sites" controls></video>
edwardloveall force-pushed el-site-glowup from 4113df07a1
Some checks failed
ci/woodpecker/pr/ci Pipeline failed
to 82c0d4468e
Some checks failed
ci/woodpecker/pr/ci Pipeline failed
2026年05月27日 21:07:10 +02:00 Compare
edwardloveall force-pushed el-site-glowup from 82c0d4468e
Some checks failed
ci/woodpecker/pr/ci Pipeline failed
to 09246418e5
All checks were successful
ci/woodpecker/pr/ci Pipeline was successful
2026年05月27日 21:53:16 +02:00 Compare
jsilasbailey left a comment
Copy link

The only thing I noticed was what seemed to be a small blip in the recording when submit is clicked before the modal closes

Don't have time to pull down and debug but it seems like a pretty small issue given this isn't submitting data
🚢

The only thing I noticed was what seemed to be a small blip in the recording when submit is clicked before the modal closes Don't have time to pull down and debug but it seems like a pretty small issue given this isn't submitting data 🚢
@ -0,0 +10,4 @@
private
def filter_by_active(scope)
First-time contributor
Copy link

nit: is this more filter_by_status?

nit: is this more `filter_by_status`?
Author
Owner
Copy link

The only thing I noticed was what seemed to be a small blip in the recording when submit is clicked before the modal closes

Yes, I see that too. The cause might be because we're patching over a gap in hotwire. I just tried it locally and it does not do that. Maybe it will over a remote network, but yeah, I'm okay with that.

> The only thing I noticed was what seemed to be a small blip in the recording when submit is clicked before the modal closes Yes, I see that too. The cause might be because we're [patching over a gap in hotwire](https://codeberg.org/rootable/tomato/commit/fed33ef94dabe0007453e488f7e0e6c7a2b10ac6). I just tried it locally and it does not do that. Maybe it will over a remote network, but yeah, I'm okay with that.
edwardloveall force-pushed el-site-glowup from 09246418e5
All checks were successful
ci/woodpecker/pr/ci Pipeline was successful
to 23ab84d03c
Some checks failed
ci/woodpecker/pr/ci Pipeline failed
2026年06月01日 16:50:49 +02:00 Compare
edwardloveall changed target branch from el-site-monthly-totals to main 2026年06月01日 17:04:10 +02:00
Author
Owner
Copy link

I think codeberg CI is struggling a bit right now, but tests passed locally for me so I'm gonna merge.

I think codeberg CI is struggling a bit right now, but tests passed locally for me so I'm gonna merge.
edwardloveall deleted branch el-site-glowup 2026年06月01日 17:07:21 +02:00
Sign in to join this conversation.
No reviewers
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
rootable/tomato!163
Reference in a new issue
rootable/tomato
No description provided.
Delete branch "el-site-glowup"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?