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 message": Allow custom communication type #1023

Open
mal-tee opened this issue Jan 30, 2023 · 1 comment
Open

"Add message": Allow custom communication type #1023

mal-tee opened this issue Jan 30, 2023 · 1 comment
Labels

Comments

@mal-tee
Copy link
Member

mal-tee commented Jan 30, 2023

in the "add message" modal of /my-requests/ we currently only support the three hard coded communication types. But sometimes companies communicate via internal messages, e.g. DMs or messaging systems behind a login. To track those accurately we should support custom values.

image

@mal-tee mal-tee added the UI/UX label Jan 30, 2023
@baltpeter
Copy link
Member

I happened to just touch the code related to this, so I thought a bit about this.

Do you think we really need to allow custom values or wouldn't it be enough to add a new value for the case you're describing?

The modal is using the same TransportMediumChooser that is also used in the generators. Adding support for custom values would be really annoying. Tbh, even just adding another static value wouldn't be fun. It would however be really easy to add the new "web form" transport medium there.

What do you think? That certainly wouldn't be perfect but otherwise someone else will have to implement this. :D

@baltpeter baltpeter mentioned this issue Feb 13, 2023
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

No branches or pull requests

2 participants