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.

Sign up for GitHub

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

Jump to bottom

Deprecate old database references #935

Open
araujoarthur0 opened this issue Jan 21, 2023 · 3 comments · May be fixed by #1053
Open

Deprecate old database references #935

araujoarthur0 opened this issue Jan 21, 2023 · 3 comments · May be fixed by #1053
Assignees
@thamara
Labels
enhancement New feature or request
Milestone
Next version

Comments

@araujoarthur0
Copy link
Collaborator

It's been some time since the migration. We can probably remove references from the old database.

time-to-leave/js/import-export.js

Line 157 in df2cb01

// TODO: 'regular' is still here while we allow importing old DB data. Please remove on the next release.

@araujoarthur0 araujoarthur0 added the enhancement New feature or request label Jan 21, 2023
@tupaschoal
Copy link
Collaborator

Should we do it for the upcoming release or just after?

@araujoarthur0
Copy link
Collaborator Author

Ah, since we've come so long without doing it and we want to release soon, I say let's wait a bit more.

@thamara
Copy link
Owner

thamara commented Jan 8, 2024

We can aim on addressing it on the next release, what do you think?

@thamara thamara self-assigned this Jan 8, 2024
@thamara thamara linked a pull request Jan 8, 2024 that will close this issue
Open
@thamara thamara added this to the Next version milestone Jan 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@thamara thamara

Labels
enhancement New feature or request
Projects
None yet
Milestone
Next version
Development

Successfully merging a pull request may close this issue.

Deprecate old db thamara/time-to-leave
3 participants
@thamara @tupaschoal @araujoarthur0