search

mslfh / xt_server

1 stars 4 forks source link

Back-end insert the user email instead of ID when inserting new exercise log data #25

Open ErikApoyy opened 6 days ago

ErikApoyy commented 6 days ago

Current behaviour: When user leave the repetition log page, it will automatically insert new data to the exercise log table. However, it takes the user email instead of the user ID. Example:

Screenshot 2024-11-26 at 7 51 38 pm

Expected behaviour: It should insert actual user ID, not their email address.

Issue: When user change their email address, all the data stored with their previous email address will be hard to track.

PatrikNguyen commented 2 hours ago

@ErikApoyy Hi Erik, could you provide me more information about the bug, which is included:

anjia1991 commented 1 hour ago

@ErikApoyy Hi Erik, could you provide me more information about the bug, which is included:

  • Steps to get that page (page contain bug) (this one will be effective for me in some other complex bug).
  • And the Url for that page. Thank you in advance.
  1. register on the system: https://app.exertime.tech/
  2. login adminjs (this is a bug related to quasar)
  3. Go Exercise player and do one exercise
  4. Click number or stop the timer. (this is another UI issue)
  5. Here is an example: https://www.exertime.tech/admin/resources/exercise_log/records/10/show image

https://www.exertime.tech/admin/resources/exercise_log/records/10/show image

ErikApoyy commented 1 hour ago

@PatrikNguyen Hi Phat,

Screenshot 2024-12-03 at 11 09 50 am Screenshot 2024-12-03 at 11 10 54 am

It's in that page. Once user click the 'Go Back' button, the POST request automatically send to the back-end (See the 2nd picture where I highlight the request).

Also, if you look at Anjia's screenshot, it's how the data being stored in the AdminJS.

Let me know if something still unclear for you, Cheers