Skip to content

Revert "Patch 2 fix: remove inline button handlers and centralize UI click binding (#627)" - #656

Merged
BaseMax merged 1 commit into
mainfrom
revert-649-patch-2
Oct 8, 2026
Merged

BaseMax merged 1 commit into
mainfrom
revert-649-patch-2

Conversation

@BaseMax

@BaseMax BaseMax commented Oct 8, 2026

Copy link
Copy Markdown
Member

Reverts #649

@BaseMax
BaseMax requested a review from jbampton as a code owner October 8, 2026 18:01
@BaseMax
BaseMax merged commit 2f0945a into main Oct 8, 2026
15 of 17 checks passed
@BaseMax
BaseMax deleted the revert-649-patch-2 branch October 8, 2026 18:01
@deepsource-io

deepsource-io Bot commented Oct 8, 2026 •

Copy link
Copy Markdown

DeepSource Code Review

We reviewed changes in 0eed0f7...0161395 on this pull request. Below is the summary for the review, and you can see the individual issues we found as inline review comments.

See full review on DeepSource ↗

PR Report Card

Overall Grade   Security  

Reliability  

Complexity  

Hygiene  

Code Review Summary

Analyzer Status Updated (UTC) Details
JavaScript Oct 8, 2026 6:01p.m. Review ↗
Secrets Oct 8, 2026 6:01p.m. Review ↗

Important

AI Review is run only on demand for your team. We're only showing results of static analysis review right now. To trigger AI Review, comment @deepsourcebot review on this thread.

@github-actions

github-actions Bot commented Oct 8, 2026

Copy link
Copy Markdown

Visit the preview URL for this PR (updated for commit 0161395):

https://nextcommunity-eea10--pr656-revert-649-patch-2-j95sg5pd.web.app

(expires Thu, 15 Oct 2026 18:02:17 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: ba5543968e5b4f067b5810ac857f9362d1b2a323

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

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant