Skip to content
This repository has been archived by the owner on Jul 22, 2022. It is now read-only.

cwp event subscriber #50

Draft
wants to merge 12 commits into
base: master
Choose a base branch
from
Draft

cwp event subscriber #50

wants to merge 12 commits into from

Conversation

softcheetah
Copy link

No description provided.

@softcheetah softcheetah requested a review from zakhap March 25, 2021 18:59
Copy link
Contributor

@zakhap zakhap left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For the most part, this boilerplate looks great! There are just a few things that need to be added in as well.

If you can, can you branch off zak.cwp and then open a new PR with the new additions that merges into this branch? Thanks!

// Project.sol Events
// TODO: Replace with actual event name when the contracts are ready
case 'DepositProject': return EventKind.DepositProject;

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@softcheetah

Can you fill in the rest of these?

description: 'A Deposit was made to a Project.',
};
}
// TODO: All events
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you add the Project events here too?

@softcheetah softcheetah marked this pull request as draft March 26, 2021 01:54
Added CWP events to `type_parse.ts` and `titler.ts`
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants