-
Notifications
You must be signed in to change notification settings - Fork 165
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
HCB Application Form: Deter use of shared email addresses #1079
Comments
Hey! I would love to work on this issue too @garyhtou. |
Go for it! Here is the updated copy that we use on HCB
|
Hey @garyhtou, can you comment on the links of the above page where the warning must be added and the source page where it is already being implemented? |
Hi @Chay2203! Warning should be implemented here site/components/fiscal-sponsorship/apply/personal-form.js Lines 26 to 33 in 905b846
The existing implementation is unfortunately in a closed source codebase. |
Sure, works! |
Here's the closed-source implementation. You can use it as a guide, however, the implementation we want here will likely be different. <div x-data="{ email: '' }">
<div class="field flex items-end">
<div class="mr2 flex-auto">
<%= form.label :email %>
<%= form.email_field :email, "x-model": "email" %>
</div>
<%= form.submit "Send" %>
</div>
<template x-if="/^(team|webmaster|marketing|admin|info|about|support|sales|hq|hello)@/.test(email)">
<div class="warning mt1">⚠️ For security, we discourage team emails on HCB. We recommend individual emails such as <em>ben@hackclub.com</em>!</div>
</template>
</div> ^ This implementation uses Alpine.js |
Sure, will look at it |
We're planning to move this form in house @ https://github.com/hackclub/hcb |
We already deter users from inviting shared email addresses on HCB.
We should do the same with the application form. This is because emails collected via the form are automatically invited to HCB.
The text was updated successfully, but these errors were encountered: