Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Jean/cow 399 placeholder add weiroll on create vesting hook #32

Merged
merged 21 commits into from
Oct 11, 2024
Merged

Jean/cow 399 placeholder add weiroll on create vesting hook #32

merged 21 commits into from
Oct 11, 2024

Conversation

JeanNeiverth
Copy link
Contributor

@JeanNeiverth JeanNeiverth commented Oct 10, 2024 •
edited
Loading

  • Add "vest all from swap" option to user
  • Resolve hooksInfo hook logic so both weiroll and userInput can work

Still todo:

  • change the useForm logic in a way amount become optional

Does not include:

  • ready UI

Temporary UI IS:
image

JeanNeiverth and others added 16 commits October 4, 2024 17:22
@JeanNeiverth
chore: use IFrame context in create-vesting
d08c224
@JeanNeiverth
feat: encode claim vesting hook with hardcoded token = wxdai
bea91c2
@JeanNeiverth
feat: get real buying token from swap in create-vesting
ee0ed92
@JeanNeiverth
feat: add llama contract mapping by chainId
a77121a
@JeanNeiverth
chore: move Spinner, WaitingSignature and SignatureSteps from withdra…
6ff0304 
…w-pool to cow-hooks-ui
@JeanNeiverth
wip: permitting token in create-vesting hook
18f7f27
@JeanNeiverth
chore: prefix UI possible review issues
90169bc
@JeanNeiverth
fix: small number token input create-vesting
df30f48
@JeanNeiverth
chore: remove cowswap generate permit function
c966b37
@JeanNeiverth
chore: remove console.logs
b0c7588
@JeanNeiverth
fix: flow to make permittable tokens work in claim vesting
471e1f5
@JeanNeiverth
fix: make claim vesting work again for not permittable tokens
d08c87d
@JeanNeiverth
wip: add weiroll
e818a2f
@yvesfracari
wip: add weiroll encoding
a3fc01b
@JeanNeiverth
feat: add logic to always use delegate call in cowShed when its a wei…
4650336 
…roll
@JeanNeiverth
feat: make both user input and weiroll work at the same time
f7b76c3
@linear Linear
Copy link

linear bot commented Oct 10, 2024

COW-399 Add weiroll on create vesting hook

@vercel Vercel
Copy link

vercel bot commented Oct 10, 2024 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
cow-hooks-dapps-claim-vesting ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 11, 2024 6:30pm
cow-hooks-dapps-create-vesting ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 11, 2024 6:30pm
cow-hooks-dapps-withdraw-pool ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 11, 2024 6:30pm

yvesfracari
yvesfracari approved these changes Oct 11, 2024
View reviewed changes
apps/create-vesting/src/app/page.tsx Outdated Show resolved Hide resolved
apps/create-vesting/src/app/page.tsx Outdated Show resolved Hide resolved
apps/create-vesting/src/app/page.tsx Outdated Show resolved Hide resolved
apps/create-vesting/src/app/page.tsx Show resolved Hide resolved
apps/create-vesting/src/app/signing/page.tsx Outdated Show resolved Hide resolved
apps/create-vesting/src/hooks/useReadTokenContract.ts Outdated Show resolved Hide resolved
apps/create-vesting/src/utils/isERC20Permit.ts Outdated Show resolved Hide resolved
apps/create-vesting/src/utils/schema.ts Show resolved Hide resolved
packages/cow-hooks-ui/src/ui/PeriodWithScaleInput.tsx Outdated Show resolved Hide resolved
packages/utils/transactionFactory/weiroll/index.ts Outdated Show resolved Hide resolved
@JeanNeiverth JeanNeiverth merged commit d4dc945 into main Oct 11, 2024
5 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yvesfracari yvesfracari yvesfracari approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@JeanNeiverth @yvesfracari