Skip to content

Conversation

@vieiro
Copy link
Contributor

@vieiro vieiro commented Sep 30, 2024

This is the proposed implementation for the discussion at #486.

@AlbertShown
Copy link
Member

Great PR! Thank you 👍

One minor change and will be ready to merge:
@example webui_set_file_handler needs to be @example webui_set_file_handler_window.

@vieiro
Copy link
Contributor Author

vieiro commented Oct 1, 2024

@AlbertShown so do we want an example for webui_set_file_handler_window too? I can create one as well...

@vieiro
Copy link
Contributor Author

vieiro commented Oct 1, 2024

Updated the @example for webui_set_file_handler_window`!

@AlbertShown
Copy link
Member

No, not line 433
but line 446!

Sorry, I wasn't clear enough the first time

@vieiro
Copy link
Contributor Author

vieiro commented Oct 1, 2024

D'oh! Very true! I added the @example to the wrong comment.
Do you want me to squash all commits together?

@AlbertShown
Copy link
Member

I guess it's okay, this PR can be merged (With typo), and we will fix the typo later 👍

@hassandraga hassandraga merged commit f8e0a20 into webui-dev:main Oct 6, 2024
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.

3 participants