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

Make Skills attr readonly for ptime attrs | PuzzleTime Sync #4 #733

Open
6 tasks
kcinay055679 opened this issue Jun 18, 2024 · 0 comments
Open
6 tasks

Make Skills attr readonly for ptime attrs | PuzzleTime Sync #4 #733

kcinay055679 opened this issue Jun 18, 2024 · 0 comments
Labels
plan needs to be planned

Comments

@kcinay055679
Copy link
Collaborator

kcinay055679 commented Jun 18, 2024

Introduction
After mapping every attribute we need from the PuzzleTime we will need to display them and make them read-only. This is necessary since PuzzleTime will be our source of truth. This results in only beeing able to edit these attributes in the PuzzleTime.

Additionally we want to have a visual feature for the user. This feature should show the user why he can't edit these fields. We also want to have a "manual sync" button to trigger the job manually and fetch the data.

ToDo

  • Look at all the attributes we get from PuzzleTime (should be documented in Look at API | PuzzleTime Sync #1 #732)
  • Make all these attributes read-only so PuzzleTime is and stays the source of truth
  • Design some kind of visual feature to mark the non-editable attributes and show it to robin to gain his approval
  • Implement this viusal feature preferably with a hoverable text that says where it's managed or why it's not editable
  • Last but not least, make sure to implement the "manual sync" button
  • Tests
@ManuelMoeri ManuelMoeri changed the title Make Skills attr readonly for ptime attrs Make Skills attr readonly for ptime attrs | PuzzleTime Sync #3 Jun 18, 2024
@RandomTannenbaum RandomTannenbaum changed the title Make Skills attr readonly for ptime attrs | PuzzleTime Sync #3 Make Skills attr readonly for ptime attrs | PuzzleTime Sync #4 Jun 18, 2024
@Robin481 Robin481 added the plan needs to be planned label Jun 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
plan needs to be planned
Projects
None yet
Development

No branches or pull requests

2 participants