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

[ENHANCEMENT]: Keep track of partially entered data in Timesheets #514

Open
1 task done
tongpu opened this issue Nov 6, 2024 · 2 comments
Open
1 task done

[ENHANCEMENT]: Keep track of partially entered data in Timesheets #514

tongpu opened this issue Nov 6, 2024 · 2 comments
Labels
enhancement This issue or pull request discusses non feature changes frontend This issue or pull request is frontend related

Comments

@tongpu
Copy link
Member

tongpu commented Nov 6, 2024

Your proposed enhancement

When I start to enter some entry in the Timesheet tab and switch to e.g. the Activity/Attendence to check something then the entry I started in the Timesheet is lost, because it was not sent to the backend.
It would be great if the frontend could somehow temporarily store this information.

Enhancement Impact

Propably adds some complexity to the frontend. 😬

Additional context

No response

Required

  • I have checked past issues to ensure that this isn't a duplicate.
@tongpu tongpu added the enhancement This issue or pull request discusses non feature changes label Nov 6, 2024
@c0rydoras
Copy link
Collaborator

If i understand this correctly, the frontend should store the currently entered report? like when its not saved?

should be possible by saving it to localStorage AFAICT

/cc @adfinis/dev-timed

@c0rydoras c0rydoras added the frontend This issue or pull request is frontend related label Nov 6, 2024
@c0rydoras c0rydoras added this to timed Nov 6, 2024
@derrabauke
Copy link
Contributor

If i understand this correctly, the frontend should store the currently entered report? like when its not saved?

should be possible by saving it to localStorage AFAICT

/cc @adfinis/dev-timed

localstorage sounds overkill to me. Simply createRecord and reuse this one when entering the timesheet fetching for the first record matching isNew.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement This issue or pull request discusses non feature changes frontend This issue or pull request is frontend related
Projects
Status: No status
Development

No branches or pull requests

3 participants