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

Add a new step to accept recipient email for the recipient registration process #516

Open
yuetloo opened this issue Jun 22, 2022 · 0 comments · Fixed by #539
Open

Add a new step to accept recipient email for the recipient registration process #516

yuetloo opened this issue Jun 22, 2022 · 0 comments · Fixed by #539
Assignees

Comments

@yuetloo
Copy link
Collaborator

yuetloo commented Jun 22, 2022

Problem:
If the environment variable, VUE_APP_GOOGLE_SPREADSHEET_ID, is defined, the optimistic recipient registration process will require the user to input an email address, and the email address is not recorded in the recipient registry contract.

This flow is broken with the introduction of the universal recipient registry which uses recipient metadata from an external source. Because the email is not available in the recipient metadata, a new page/step is needed in the recipient registration process to take the user email address when VUE_APP_GOOGLE_SPREADSHEET_ID is defined.

@auryn-macmillan , VUE_APP_GOOGLE_SPREADSHEET_ID, seems to be a new feature introduced post v0.6.0, please confirm if this feature should be retained.

@yuetloo yuetloo changed the title Add a new step to accept recipient email on the recipient registration process Add a new step to accept recipient email for the recipient registration process Jun 22, 2022
@auryn-macmillan auryn-macmillan added this to the Current milestone Jun 23, 2022
@yuetloo yuetloo linked a pull request Aug 3, 2022 that will close this issue
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 a pull request may close this issue.

2 participants