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

feat: bug reporting by user on the app #324

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

tomaspalma
Copy link
Member

Closes #322.

Adds a form where users can submit bugs.

The design is somewhat experimental, if you don't like it, please say so!

@tomaspalma tomaspalma requested a review from a team October 11, 2024 15:20
@tomaspalma tomaspalma self-assigned this Oct 11, 2024
Copy link

netlify bot commented Oct 11, 2024

Deploy Preview for tts-fe-preview ready!

Name Link
🔨 Latest commit 46916fe
🔍 Latest deploy log https://app.netlify.com/sites/tts-fe-preview/deploys/6736386ea63a530008af7f19
😎 Deploy Preview https://deploy-preview-324--tts-fe-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@thePeras
Copy link
Member

We want to be limited to just bug reports? Why not suggestions and other types of feedbacks?

@tomaspalma
Copy link
Member Author

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

@tomaspalma tomaspalma force-pushed the feature/bug-reporting branch 3 times, most recently from edd7bf1 to 4d362bd Compare October 12, 2024 22:12
@tomaspalma
Copy link
Member Author

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.

Copy link
Member

@thePeras thePeras left a 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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add way for users to report bugs
2 participants