-
Notifications
You must be signed in to change notification settings - Fork 1
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
feat: bug reporting by user on the app #324
base: develop
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for tts-fe-preview ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
d15d3e3
to
5740e3e
Compare
We want to be limited to just bug reports? Why not suggestions and other types of feedbacks? |
You're right! I will add a dropdown for the users to choose whether or not they want to report a bug or suggest feedback |
edd7bf1
to
4d362bd
Compare
Already changed the position of the bug report on desktop but left it at the original place on mobile. Also added ability for user to submit bug or suggestion. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a really nice new thing. We can next config a slack integration if we want to.
Some things I notice:
- It shouldn't allow submitting empty feedback.
- Can we change the icon? I thinkg the Spider is not the most adequate icon
- The button can be full rounded (circle)?
- When opened, the modal can have a left margin?
- Clear the inputs after submitting
- Add some "Enviado" in the "obrigado" phrase, so that the user knows it was submitted
Also, I think netlify does not have the sentry variables and the feedback is not beeing submited, besides it displays "Obrigado". Maybe a try catch is necessary somewhere
4d362bd
to
46916fe
Compare
Closes #322.
Adds a form where users can submit bugs.
The design is somewhat experimental, if you don't like it, please say so!