-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add support for proxy lists #22
Conversation
6d4ebdb
to
00d39c6
Compare
You need to format your commit appropriately so that semantic release can trigger a new release, update the change log and tag it on github for you. |
Looks reasonable otherwise. |
Please rework/reword your commit for semantic release |
Support proxy lists in the same way as string/number lists
0b0c9a0
to
38a0533
Compare
@Jo-Byr can you add an example? |
I only implemented the proxy list support in the vue2 part of trame-simput. Maybe should I convert this PR to a draft PR for the time being and once the vue2 changes are validated I can add them to the vue3 part ? |
Fix bug making it impossible to add a proxy to an empty proxy list
Should we merge? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems reasonable
Support proxy lists in the same way as string/number lists