-
Notifications
You must be signed in to change notification settings - Fork 0
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
Create a webform to create PRs #3
Comments
We could also offer a dropdown of existing entries to be updated. |
concerning the updates: |
Yes, but what if someone entered a wrong e-mail once and cannot update it anymore? And if we simply believe someone saying "I cannot access the original e-mail anymore", then what additional security does that validation token bring us? I think having no authentication but peer review on Github is probably the more pragmatic approach. |
@dbrgn the original email adress is in the json file provided from your server. So you should be able to change it. As long as you own the Domain, it is possible. |
Something like this?
|
👍 I'd suggest implementing the form with direct PR creation first though, and only adding verification once we start having problems with that approach. |
+1 for the webform and direct PR, anybody in a hackerspace should be able to create and update the url, not only the one controlling the email/server. People will probably loss acces to the email, then we'd need a mechanism to re-authenticate the space url, which seems complicated. |
The website would present a form for new entries.
Once the new entry is validated and checked against duplicates the users request will be translated into a pull request against the current master of the database repo.
The text was updated successfully, but these errors were encountered: