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

Telephone numbers #138

Open
mhvis opened this issue May 9, 2019 · 2 comments · May be fixed by #197 or #231
Open

Telephone numbers #138

mhvis opened this issue May 9, 2019 · 2 comments · May be fixed by #197 or #231
Labels
enhancement New feature or request

Comments

@mhvis
Copy link
Member

mhvis commented May 9, 2019

Feature request from someone else: allow telephone numbers in the system so that people can be stalked about dinner and payment.

If we make it optional and note down that everyone can see it if you fill it in it should be reasonably fine for privacy reasons I think.

@DutcherNL
Copy link
Collaborator

Another option, though extensive one, is to implement chats in the system. It puts an additional load on the server, but ensures data privacy (this becomes useful when users do not want to set their phone number in the app.

@mhvis
Copy link
Member Author

mhvis commented May 12, 2019

Chat is an option but depends on a notification system which is not yet in place. As the primary goal of the app in this case is to facilitate communication between diners we should probably primarily go for the channels that the users are most familiar with. So I have the following suggestion for in a new PR:

  • Add telephone field (with a helpful privacy notice as help_text)
  • Add option for user to make e-mail address public
  • Find a way of displaying it in the dining entries page, I'm thinking of using a javascript expand behind a button

I don't think that there will be a lot of users with privacy concerns but if that's the case then that's fine.

@mhvis mhvis added the enhancement New feature or request label Oct 11, 2019
@mhvis mhvis linked a pull request Oct 28, 2021 that will close this issue
@mhvis mhvis linked a pull request Dec 27, 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
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants