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

Add lastAccessed field to map table #286

Open
3 tasks
JannikStreek opened this issue Jun 17, 2024 · 0 comments · May be fixed by #296
Open
3 tasks

Add lastAccessed field to map table #286

JannikStreek opened this issue Jun 17, 2024 · 0 comments · May be fixed by #296
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@JannikStreek
Copy link
Member

JannikStreek commented Jun 17, 2024

Proposed feature
We would like to standardize our deletion policies of old data across our open source apps. To make it as easy as possible, we have decided to leverage the last access of a created entity. Currently teammapper does not have such a field on the map entity and we use the lastModified field.

  • Add a lastAccessed timestamp field to map entity
  • The field should get updated when findOne is called in the controller => Please check that this method is actually called when accessing a map, I think it should be called.
  • Adapt the deletion algorithm in the map service to use this field instead of lastModified and as a backup the old field should be used (as many legacy maps don't have any data inside the new field).
@JannikStreek JannikStreek added enhancement New feature or request good first issue Good for newcomers labels Jun 17, 2024
@sorenjohanson sorenjohanson linked a pull request Jun 20, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant