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

fix: polygon support #161

Merged
merged 7 commits into from
Sep 2, 2024
Merged

fix: polygon support #161

merged 7 commits into from
Sep 2, 2024

Conversation

HJunyuan
Copy link
Member

@HJunyuan HJunyuan commented Aug 30, 2024 •
edited
Loading

Context

  • /email and /verify does not work for Polygon-issued documents

What does this PR do?

  • Create a shared verify() to be used by the various lambda functions
  • Auto switch between Ethereum and Polygon dependent on document.network field (see reference implementation)

@HJunyuan
chore: bump oa libraries
ab363db
@HJunyuan
fix: verify should auto-switch between ethereum and polygon
129bebe 
- caveat: `src/storage/documentService/documentService.js` can't seem to use `src/shared/verify.js` properly
@HJunyuan
fix: use shared verify for documentService.js
11b06af
@seed-deploy Seed Deploy
Copy link

seed-deploy bot commented Aug 30, 2024

View stack outputs

@HJunyuan HJunyuan marked this pull request as ready for review August 30, 2024 09:09
phanshiyu
phanshiyu reviewed Aug 30, 2024
View reviewed changes
src/shared/verify.js Outdated Show resolved Hide resolved
@seed-deploy seed-deploy bot temporarily deployed to pr161 August 31, 2024 11:44 Inactive
@HJunyuan HJunyuan merged commit 5ea1c1a into master Sep 2, 2024
6 checks passed
@HJunyuan HJunyuan deleted the fix/polygon-support branch September 2, 2024 06:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@phanshiyu phanshiyu phanshiyu 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
@HJunyuan @phanshiyu