Skip to content
This repository has been archived by the owner on Dec 17, 2021. It is now read-only.

Add support for ERC20 tokens #70

Merged
merged 8 commits into from
Dec 2, 2021
Merged

Add support for ERC20 tokens #70

merged 8 commits into from
Dec 2, 2021

Conversation

QEDK
Copy link
Member

@QEDK QEDK commented Nov 17, 2021

  • Overhauls contract flow to only use ERC20 tokens
  • Add support in frontend for approvals

Fixes #21

@QEDK QEDK added the enhancement New feature or request label Nov 17, 2021
@QEDK QEDK requested a review from Cali93 November 17, 2021 18:14
@QEDK QEDK self-assigned this Nov 17, 2021
@QEDK QEDK mentioned this pull request Nov 29, 2021
@Dhaiwat10
Copy link
Member

Dhaiwat10 commented Dec 1, 2021

Added a temp modal that asks for stake amount to recruiters when they click on the request information button. The request and approveRequest functionality is working fine.

One problem: For some reason, now I cannot unlock my own information. Well, I can unlock it but I can't see it. The backend returns an error. This was possible earlier. I did both the transactions successfully - request and approve request. Not sure if that's by design but just putting it out here.

Other notes: Had to rebase this branch against staging since this branch didn't have the latest frontend code.

cc @QEDK @Cali93

packages/react-app/components/cards/MediaCard.jsx Outdated Show resolved Hide resolved
packages/react-app/components/cards/MediaCard.jsx Outdated Show resolved Hide resolved
packages/react-app/components/cards/MediaCard.jsx Outdated Show resolved Hide resolved
packages/react-app/components/cards/MediaCard.jsx Outdated Show resolved Hide resolved
@Dhaiwat10
Copy link
Member

Dhaiwat10 commented Dec 1, 2021 via email

@QEDK QEDK marked this pull request as ready for review December 2, 2021 14:27
@QEDK QEDK merged commit be30a6d into staging Dec 2, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add support for ERC20 token
2 participants