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.

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

Tweak scripts to support multiple token owners and transfers #26

Merged
merged 2 commits into from
Aug 1, 2022

Conversation

dave
Copy link
Contributor

@dave dave commented Jul 26, 2022

No description provided.

@dave
Copy link
Contributor Author

dave commented Jul 26, 2022

Hey @pheuberger here's some changes that we need to support the full end-to-end tests (with reverse bridge direction)... I don't think these will break anything in this repo!

@dave
Copy link
Contributor Author

dave commented Jul 26, 2022

It will probably need some rebasing into your existing PR, but I thought I'd put it here so we know what's needed.

@dave
Copy link
Contributor Author

dave commented Jul 26, 2022

@pheuberger
Copy link
Collaborator

should I review this already or do you want to rebase first? :)

@dave
Copy link
Contributor Author

dave commented Aug 1, 2022

I'll rebase today.

@dave
Copy link
Contributor Author

dave commented Aug 1, 2022

OK @pheuberger that's rebased now.

@dave dave force-pushed the dave/accounts branch 2 times, most recently from ae19834 to aee1534 Compare August 1, 2022 13:43
lib/toucan.js Outdated Show resolved Hide resolved
lib/toucan.js Outdated Show resolved Hide resolved
scripts/deploy-local.js Show resolved Hide resolved
@pheuberger
Copy link
Collaborator

can you fixup your review tweaks into the commits you're fixing? :)

@dave
Copy link
Contributor Author

dave commented Aug 1, 2022

Done!

Copy link
Collaborator

@pheuberger pheuberger left a comment

Choose a reason for hiding this comment

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

awesome, thank you

@pheuberger pheuberger merged commit bc7cda7 into regen-network:master Aug 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants