Skip to content

make non-zk proofs work again#1

Open
nalinbhardwaj wants to merge 2 commits into
proofcarryingdata:masterfrom
nalinbhardwaj:nibnalin/upgrade
Open

make non-zk proofs work again#1
nalinbhardwaj wants to merge 2 commits into
proofcarryingdata:masterfrom
nalinbhardwaj:nibnalin/upgrade

Conversation

@nalinbhardwaj
Copy link
Copy Markdown

@nalinbhardwaj nalinbhardwaj commented Nov 14, 2023

upgrades and vendoring in passport interface files

screen-record-zukit.mov

Copy link
Copy Markdown
Contributor

@dcposch dcposch left a comment

Choose a reason for hiding this comment

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

LGTM after comments

Comment thread src/ZupassLoginButton.tsx
const label = state.anonymous
? text("🕶️", "Welcome, anon")
: text("👓", state.participant.name);
: text("👓", state.participant.email);
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

👍

Comment thread src/ZupassProvider.tsx
signal,
externalNullifier,
};
throw new Error("Anonymous login not implemented");
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

just curious, what broke about this part? it was very generic---you pass in a groupURL , can be any semphore group not necessarily zuzalu related

Copy link
Copy Markdown
Author

@nalinbhardwaj nalinbhardwaj Nov 14, 2023

Choose a reason for hiding this comment

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

Similar to the other one, importing stuff from the core @pcd/* packages is broken ATM (esoteric errors that are quite hard to debug). That one I fixed by vendoring, slowly commenting out stuff and finding what's broken. Will have to do the same for this but would be a lot of effort since this path had a lot of imports internally.

ideally should just fix the issues in main pcd repo instead.

Comment thread src/state.ts
status: "logged-out" | "logged-in";
anonymous?: boolean;
participant?: ZuParticipant;
participant?: User;
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

👍

Comment thread src/vendor/SemaphoreSignatureIntegration.ts
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.

2 participants