-
Notifications
You must be signed in to change notification settings - Fork 89
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(swap): display order hooks details #4925
Conversation
… feat/hooks-iframe-app # Conflicts: # apps/cowswap-frontend/src/modules/hooksStore/hooks/usePostHooksRecipientOverride.ts # apps/cowswap-frontend/src/modules/hooksStore/hooks/useSetRecipientOverride.ts # apps/cowswap-frontend/src/modules/hooksStore/types/hooks.ts # apps/cowswap-frontend/src/modules/trade/hooks/useResetRecipient.ts
…l/cowswap into feat/hooks-iframe-app-1
…col/cowswap into feat/hooks-iframe-app-2
…col/cowswap into feat/hooks-iframe-app-3
…col/cowswap into feat/hooks-iframe-app-4
…col/cowswap into feat/hooks-iframe-app-5
…col/cowswap into feat/hooks-iframe-app-6
…col/cowswap into feat/hooks-iframe-app-7
…col/cowswap into feat/hooks-iframe-app-8
… feat/hooks-iframe-app
…l/cowswap into feat/hooks-iframe-app-1 # Conflicts: # apps/cowswap-frontend/src/modules/trade/hooks/useNotifyWidgetTrade.ts
…col/cowswap into feat/hooks-iframe-app-3
…col/cowswap into feat/hooks-iframe-app-4
…col/cowswap into feat/hooks-iframe-app-6
…col/cowswap into feat/hooks-iframe-app-8
Co-authored-by: Leandro <[email protected]>
… feat/hook-to-dapp-matching
…otocol/cowswap into feat/hook-to-dapp-matching-explorer # Conflicts: # libs/hook-dapp-lib/package.json
…com/cowprotocol/cowswap into feat/hook-to-dapp-matching-cowswap
…4928) * feat(hooks-store): update custom hook-dapps manifests periodically * chore: fix memo isDarkMode * chore: fix code style
{(children) => ( | ||
<ConfirmDetailsItem label="Hooks" tooltip="Hooks are interactions before/after order execution."> | ||
{children} | ||
</ConfirmDetailsItem> | ||
)} |
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.
Similar comment to the explorer changes: fn children instead of flat JSX
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.
But we need it here in order to wrap the content into ConfirmDetailsItem
.
Also, Michel changed it a bit in his PR
… feat/hook-to-dapp-matching-cowswap # Conflicts: # apps/cowswap-frontend/src/modules/hooksStore/pure/AddCustomHookForm/CustomDappLoader/index.tsx # libs/hook-dapp-lib/src/utils.ts
Summary
Displaying of order hook details.
UI is not ready yet and will be implemented in another PR!
To Test
No need to test (please test all recent changes all together when design for hook details is ready)