Skip to content

refactor(render) :: drop the unread component name in handle_row - #1428

Merged
lovasoa merged 1 commit into
sqlpage:mainfrom
81reap:mutation/09-1-handle-row-component-check
Sep 6, 2026
Merged

refactor(render) :: drop the unread component name in handle_row#1428
lovasoa merged 1 commit into
sqlpage:mainfrom
81reap:mutation/09-1-handle-row-component-check

Conversation

@81reap

@81reap 81reap commented Sep 6, 2026

Copy link
Copy Markdown
Collaborator

handle_row looked up the current component's name and then only checked whether a component was open at all. The name was never read.

found by #1396

@81reap
81reap marked this pull request as ready for review September 6, 2026 07:06
@81reap

81reap commented Sep 6, 2026

Copy link
Copy Markdown
Collaborator Author

CI failure is unrelated to this change, it comes from this commit on main 4c60e5d

`handle_row` looked up the current component's name and then only checked whether a component was open at all. The name was never read.
@lovasoa
lovasoa force-pushed the mutation/09-1-handle-row-component-check branch from c286af9 to 6878ca8 Compare September 6, 2026 12:56
@lovasoa lovasoa changed the title fix(render) :: drop the unread component name in handle_row refactor(render) :: drop the unread component name in handle_row Sep 6, 2026
@lovasoa
lovasoa merged commit 6d32b40 into sqlpage:main Sep 6, 2026
51 checks passed
@81reap
81reap deleted the mutation/09-1-handle-row-component-check branch September 6, 2026 19:29
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