Skip to content

Add "Copy to clipboard" button to the ZMI Test tab#54

Merged
dataflake merged 2 commits intomasterfrom
codemonkey440/rendered_sql_clipboard_copy_button
Jan 17, 2026
Merged

Add "Copy to clipboard" button to the ZMI Test tab#54
dataflake merged 2 commits intomasterfrom
codemonkey440/rendered_sql_clipboard_copy_button

Conversation

@dataflake
Copy link
Member

Closes #53

The button copies the ZSQL method's rendered SQL query output to the user's clipboard to help debugging. See #53 for a screen shot showing what that looks like.

@dataflake dataflake requested review from drfho and icemac January 17, 2026 09:14
@dataflake dataflake self-assigned this Jan 17, 2026
@drfho
Copy link
Contributor

drfho commented Jan 17, 2026

hi @dataflake , button is a nice enhancement.
To make it work with one-liner-SQL I removed button-margins and monotonized borders for perfect fit.
And I made the upper part of the gui a little bit nicer, too.

zsqlmethod1

@drfho
Copy link
Contributor

drfho commented Jan 17, 2026

BTW:
On starting Zope (5.13.1) I got a ModuleNotFoundError: No module named 'Shared.DC.ZRDB' after installing this branch of Products.ZSQLMethods as editable. May be connected to the new setup. Copilot helped me to make setup.py working. I will create a PR: #55
Please delete it, if it does not match to Your ideas about the new setup. Thank for your effort.

Copy link
Contributor

@drfho drfho left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@dataflake
Copy link
Member Author

Thank you very much for taking the time for these nice improvements! See #55 about the errors you saw - you must use Zope 6 for the package to load correctly.

@dataflake dataflake merged commit e9638a6 into master Jan 17, 2026
14 checks passed
@dataflake dataflake deleted the codemonkey440/rendered_sql_clipboard_copy_button branch January 17, 2026 13:51
@dataflake
Copy link
Member Author

P.S.: Use Zope 6 and update zc.buildout and setuptools to match the current recommendations (setuptools >= 78.1.1,<81 and zc.buildout >5.1)

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.

Add copy to clipboard button to ZMI ZSQL rendered query

2 participants