coracle/flotilla
1
6
Fork 24
Code Issues 42 Pull Requests 2 Actions Packages Projects 5 Releases Wiki Activity

Room Delete partial failure #194

New Issue
Open
opened 2026-04-13 18:58:48 +00:00 by hodlbod · 6 comments
hodlbod commented 2026-04-13 18:58:48 +00:00
Owner
Copy Link
Copy Source

When you delete a room, it succeeds on the relay, but doesn't remove the room from the navigation, even after a refresh.

When you delete a room, it succeeds on the relay, but doesn't remove the room from the navigation, even after a refresh.
hodlbod added this to the Current milestone 2026-04-13 18:58:48 +00:00
hodlbod added the dev label 2026-04-13 18:58:48 +00:00
bhavishy2801 commented 2026-04-13 20:59:34 +00:00
Contributor
Copy Link
Copy Source

Hi @hodlbod. Can you assign me this issue for fixing this failure?

Hi @hodlbod. Can you assign me this issue for fixing this failure?
bhavishy2801 was assigned by hodlbod 2026-04-13 21:13:36 +00:00
bhavishy2801 commented 2026-04-14 15:41:09 +00:00
Contributor
Copy Link
Copy Source

Hi @hodlbod. I've fixed the room delete failure and raised the PR for this. Kindly review the PR :)

Hi @hodlbod. I've fixed the room delete failure and raised the PR for this. Kindly review the PR :)
bhavishy2801 commented 2026-04-14 15:52:48 +00:00
Contributor
Copy Link
Copy Source

Hi @hodlbod, I’m really sorry, it seems the PR got merged from my side unintentionally.

I was trying to squash my commits from my fork, but it looks like that action ended up merging the PR into the dev branch of the upstream. That wasn’t my intention.

Could you please let me know if I had write access, or how this might have happened? I’m happy to revert the changes or follow the correct process to fix this.

Apologies again for the confusion.

Hi @hodlbod, I’m really sorry, it seems the PR got merged from my side unintentionally. I was trying to squash my commits from my fork, but it looks like that action ended up merging the PR into the `dev` branch of the upstream. That wasn’t my intention. Could you please let me know if I had write access, or how this might have happened? I’m happy to revert the changes or follow the correct process to fix this. Apologies again for the confusion.
hodlbod reopened this issue 2026-04-14 19:06:25 +00:00
hodlbod commented 2026-04-14 19:07:41 +00:00
Author
Owner
Copy Link
Copy Source

I’m really sorry, it seems the PR got merged from my side unintentionally.

Oh, I see, no worries. I've force pushed on dev, so all good now. I left a comment on the PR, but I don't think that was the correct solution, I think it's probably more due to sync flakiness, which is going to be challenging to diagnose.

> I’m really sorry, it seems the PR got merged from my side unintentionally. Oh, I see, no worries. I've force pushed on dev, so all good now. I left a comment on the PR, but I don't think that was the correct solution, I think it's probably more due to sync flakiness, which is going to be challenging to diagnose.
hodlbod commented 2026-04-14 20:02:51 +00:00
Author
Owner
Copy Link
Copy Source

For more context, here's what I'm seeing. The room gets deleted on the relay, but the meta event gets retained while the room delete event doesn't seem to be.

For more context, here's what I'm seeing. The room gets deleted on the relay, but the meta event gets retained while the room delete event doesn't seem to be.
Screen Recording 2026-04-14 at 1.01.37 PM.mov
7.6 MiB
Khushvendra commented 2026-04-20 21:45:02 +00:00
Contributor
Copy Link
Copy Source

@hodlbod is this issue open to be worked on?

@hodlbod is this issue open to be worked on?
bhavishy2801 was unassigned by hodlbod 2026-05-06 16:35:33 +00:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
design
For issues related to design work. Development should be done in another issue and refer back to this one.
 dev
Issues related to development work
 easy
Issues that can be executed on by new contributors
 idea
For issues that aren't yet actionable
 priority
For issues related to high priority features and bugs
 usability
Issues that have to be validated with real live human beings.
No Label dev
Milestone
No items
No Milestone Current
Projects
Clear projects
No project
Assignees
Clear assignees
hodlbod (hodlbod) mplorentz (Matt Lorentz) sakshamjain (Saksham Jain) sondreb triesap userAdityaa (Aditya Chaudhary)
No Assignees
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: coracle/flotilla#194
Delete Branch "%!s()"

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?