Skip to content

Feature/private polygons#34

Open
0xPetra wants to merge 28 commits intomasterfrom
feature/private-polygons
Open

Feature/private polygons#34
0xPetra wants to merge 28 commits intomasterfrom
feature/private-polygons

Conversation

@0xPetra
Copy link
Contributor

@0xPetra 0xPetra commented Nov 21, 2022

Description

⛑️ What was done:

  • New Scaffolding and routing
  • 4, 6, and 10 public point polygons
  • 4, 6, and 10 private point polygons
  • Uploaded zkey/wasm to pinata
  • Warning about max number of polygons (10 points max)

🎫 Ticket:

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update
  • Ready to merge

🧪 How to test:

🗒️ Notes:

  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have commented my code, particularly in hard-to-understand areas
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules

Closes #(issue)

@vercel
Copy link

vercel bot commented Nov 21, 2022

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

Name Status Preview Comments Updated (UTC)
client ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 7, 2023 5:34pm

Copy link
Collaborator

@BlakeMScurr BlakeMScurr left a comment

Choose a reason for hiding this comment

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

Partial review, I still have a lot to read!

{
"indexed": true,
"internalType": "bool",
"name": "isVerifie",
Copy link
Collaborator

Choose a reason for hiding this comment

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

I noticed you fixed this in the other pr. Is this out of date?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We do need to update these files, but first, we should wait to see if we need to redeploy from:
zkMaps/zkMaps#43
zkMaps/zkMaps#44

<Route exact path="/create">
<CreateZone {...routeProps} />
</Route>
<Route exact path="/ethdenver">
Copy link
Collaborator

Choose a reason for hiding this comment

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

What's /ethdenver for? Why does it just have the same stuff as Home?

Copy link
Contributor Author

@0xPetra 0xPetra Nov 26, 2022

Choose a reason for hiding this comment

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

/ethdenver has the map we released for the hackathon.

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.

3 participants