-
Notifications
You must be signed in to change notification settings - Fork 399
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
feat(frontend): improve approval form edition + add a delete button t… #3946
Conversation
…o the EditableSchema
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.
👍 Looks good to me! Reviewed everything up to 1806526 in 38 seconds
More details
- Looked at
248
lines of code in5
files - Skipped
0
files when reviewing. - Skipped posting
1
drafted comments based on config settings.
1. frontend/src/lib/components/flows/content/FlowModuleScript.svelte:38
- Draft comment:
ThetoggleShowAll
function correctly toggles theshowAllCode
state, which is used to conditionally render the code content. This implementation is straightforward and effective for the intended functionality. - Reason this comment was not posted:
Confidence changes required:0%
The implementation of thetoggleShowAll
function inFlowModuleScript.svelte
is straightforward, toggling theshowAllCode
boolean state. This state is then used in the template to conditionally render either the full code or a truncated version with a button to show all. The logic seems correct as it directly toggles the state and the UI reacts to this state change to show/hide the full code.
Workflow ID: wflow_P8J8H35SxcJEqG3I
You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet
mode, and more.
…ndmill-labs/windmill into fr/approval-form-improvements
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.
👍 Looks good to me! Incremental review on 564e0d6 in 1 minute and 47 seconds
More details
- Looked at
77
lines of code in2
files - Skipped
0
files when reviewing. - Skipped posting
1
drafted comments based on config settings.
1. frontend/src/lib/components/flows/content/FlowModuleSuspend.svelte:219
- Draft comment:
TheToggle
component for 'JSON View' is always unchecked and disabled, which contradicts the intended functionality of allowing users to toggle JSON view mode. Consider making thechecked
anddisabled
properties dynamic based on the appropriate conditions.
<Toggle
checked={jsonView}
disabled={!flowModule.suspend}
label="JSON View"
size="xs"
options={{
right: 'JSON Editor',
rightTooltip: 'Arguments can be edited either using the wizard, or by editing their JSON Schema.'
}}
lightMode
on:change={() => {
jsonView = !jsonView;
if (flowModule.suspend) {
flowModule.suspend.resume_form = {
schema: emptySchema()
}
}
}}
/>
- Reason this comment was not posted:
Confidence of 0% on close inspection, compared to threshold of 50%.
Workflow ID: wflow_2uPJLsSVQ9Igu70g
You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet
mode, and more.
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.
👍 Looks good to me! Incremental review on d2378cc in 3 minutes and 38 seconds
More details
- Looked at
12
lines of code in1
files - Skipped
0
files when reviewing. - Skipped posting
0
drafted comments based on config settings.
Workflow ID: wflow_Xi6p6W5eDmbe78CR
You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet
mode, and more.
…ndmill-labs/windmill into fr/approval-form-improvements
…o the EditableSchema
![Screenshot 2024-06-21 at 15 08 25](https://private-user-images.githubusercontent.com/456655/341772329-fb15e681-958a-4011-a628-9bdb579a8857.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjA3MjQ3MTcsIm5iZiI6MTcyMDcyNDQxNywicGF0aCI6Ii80NTY2NTUvMzQxNzcyMzI5LWZiMTVlNjgxLTk1OGEtNDAxMS1hNjI4LTliZGI1NzlhODg1Ny5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjQwNzExJTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI0MDcxMVQxOTAwMTdaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT0yNGZhNmI4NjAxOTlkZTgwMDQ3NjYwODBmOTA1NTk5MmYwOGJlOWQzNjZmODI3MGNmYzA3OWI3ZjllZWU3NjU1JlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCZhY3Rvcl9pZD0wJmtleV9pZD0wJnJlcG9faWQ9MCJ9.9YLZfoDWBc10buSJHQBtGBwacsW_AIQK7fFEtYbhwGw)
![Screenshot 2024-06-21 at 15 08 03](https://private-user-images.githubusercontent.com/456655/341772338-d0082e59-702d-456b-837c-c16ff5aac79b.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjA3MjQ3MTcsIm5iZiI6MTcyMDcyNDQxNywicGF0aCI6Ii80NTY2NTUvMzQxNzcyMzM4LWQwMDgyZTU5LTcwMmQtNDU2Yi04MzdjLWMxNmZmNWFhYzc5Yi5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjQwNzExJTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI0MDcxMVQxOTAwMTdaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT0zMzdkM2M2ZDhkMzUzMTk2MTc5ODYxZTM3OWExOTQ1ZDUyMDY3MjkzNjI1MDM0Nzg3YmY2MzY5ZTY3NDE5NzUyJlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCZhY3Rvcl9pZD0wJmtleV9pZD0wJnJlcG9faWQ9MCJ9.wg9HNgEIOHfTCJ26HMn2XA1ObTP5V-f6nVp6O6-uZxM)
Summary:
Enhanced approval form editing and added a delete button to EditableSchema component in the frontend, including new JSON view toggling functionality.
Key points:
EditableSchemaDrawer
andAddProperty
components.EditableSchemaDrawer
to remove schema properties.jsonView
state and logic to toggle JSON view for form editing inFlowModuleSuspend.svelte
.change
event after deleting an argument inAddProperty.svelte
.Generated with ❤️ by ellipsis.dev