Skip to content

Fix CBs shouldn't see delete and edit discussions#9149

Merged
nabramow merged 1 commit into
developfrom
na/fix-delete-comment
Jun 21, 2026
Merged

Fix CBs shouldn't see delete and edit discussions#9149
nabramow merged 1 commit into
developfrom
na/fix-delete-comment

Conversation

@nabramow

Copy link
Copy Markdown
Member

Removing this as it looks like we changed the backend then I forgot to update it on the frontend. This is the bug we saw when screensharing at the last meeting.

Testing

Explain how you tested this PR and give clear steps so the reviewer can replicate.

Web frontend checklist

  • There are no console warnings when running the app
  • Added tests where relevant
  • Clicked around my changes running locally and it works
  • Checked Desktop, Mobile and Tablet screen sizes

For maintainers

  • Maintainers can push commits to my branch
  • Maintainers can merge this PR for me

@CouchersBot

CouchersBot commented Jun 20, 2026

Copy link
Copy Markdown
Member

Mobile

Install the Dev Tool (iOS via TestFlight, iOS Simulator, or Android .apk) here.

Scan the QR with your phone camera, or tap Open in Dev Tool on the device, to open this branch in the installed Dev Tool dev client.

    iOS         Android    
    QR to open the iOS build         QR to open the Android build    
    Open in Dev Tool         Open in Dev Tool    
Deep links

iOS

couchers-devtool://expo-development-client/?url=https%3A%2F%2Fa0381b6e--ota.preview.couchershq.org%2Fios%2Fmanifest

Android

couchers-devtool://expo-development-client/?url=https%3A%2F%2Fa0381b6e--ota.preview.couchershq.org%2Fandroid%2Fmanifest

Web (Vercel)

View the Vercel web preview for this branch.

Backend

Schema Schema diff Sample emails

Other

Protos Backend coverage Web coverage

@nabramow
nabramow merged commit 55d91f3 into develop Jun 21, 2026
8 checks passed
@nabramow
nabramow deleted the na/fix-delete-comment branch June 21, 2026 09:12
@github-actions

Copy link
Copy Markdown
Contributor

📝 Release Notes

This PR does not need to be included in release notes.

Reason: This fixes a permissions/UI bug in community discussions where community builders or moderators were incorrectly shown edit/delete options. While user-facing, it appears to be a relatively minor bug affecting a specific role and not broad or critical enough for release notes.

🤖 Bot Debug Information

Model: couchers.openai.gpt-5.4
Decision: exclude
Reasoning: This fixes a permissions/UI bug in community discussions where community builders or moderators were incorrectly shown edit/delete options. While user-facing, it appears to be a relatively minor bug affecting a specific role and not broad or critical enough for release notes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants