Skip to content

feat(logs): add small interface to submit tickets - #169

Open
afymer wants to merge 8 commits into
Telecom-Etude:mainfrom
afymer:log-user-activity
Open

feat(logs): add small interface to submit tickets#169
afymer wants to merge 8 commits into
Telecom-Etude:mainfrom
afymer:log-user-activity

Conversation

@afymer

@afymer afymer commented Aug 29, 2025

Copy link
Copy Markdown
Collaborator

Description

Adds a small floating button in the bottom right of the screen that opens a dialog to send a ticket.
This also adds a hook and a method to log data on client side, to be joined with the ticket.

Related Issue(s)

RGPD is quite an issue here but don't wory ~

Type of Change

  • New feature
  • Bug fix
  • Chore
  • Documentation update
  • Performance upgrade

Checklist

  • My code follows the style guidelines.
  • All tests pass.
  • The code is documented (if applicable).

@afymer afymer self-assigned this Sep 2, 2025
@afymer afymer changed the title feat(logs): add small interface to subtmit tickets feat(logs): add small interface to submit tickets Sep 14, 2025
@t-webber

Copy link
Copy Markdown
Collaborator

The button already exists in the top right hand size corner, why add another? I also prefer the style of the previous one.

Concerning the storage, I am not sure storing the bug reports in Prisma is interesting, as all we want is a notification, no? Why don't you send an email to dev@telecom-etude.fr with the content of the bug report instead?

@t-webber

t-webber commented Sep 21, 2025

Copy link
Copy Markdown
Collaborator

also it would have been easier to separate the commits, it's weird to change the formatting of Prisma here as well. Maybe we should add a CI with prisma format? And a .vscode/settings item or .editorconfig for users that use fancy IDEs that don't use the official prisma fmt

@t-webber t-webber left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cf previous messages

@afymer

afymer commented Sep 23, 2025

Copy link
Copy Markdown
Collaborator Author

The button already exists in the top right hand size corner, why add another? I also prefer the style of the previous one.

Concerning the storage, I am not sure storing the bug reports in Prisma is interesting, as all we want is a notification, no? Why don't you send an email to dev@telecom-etude.fr with the content of the bug report instead?

You are absolutely right, an email is better.
The distinction with the existing button lies in the purpose of this button, which was first intended only for the early days of deployment, but after thinking about it I don't see any reason not to keep it afterwards.

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.

2 participants