-
-
Notifications
You must be signed in to change notification settings - Fork 365
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
Cleanup Database
interface
#6119
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh, you are? That's cool!
By the way, there is probably little reason to keep the Database
interface when the database dealing stuff has been migrated to that multiplatform-enabled thing. (We'll see)
app/src/main/java/de/westnordost/streetcomplete/data/AndroidDatabase.kt
Outdated
Show resolved
Hide resolved
But we will need to keep the |
This reverts commit 7ba84ba.
You are probably right about this. The interface of that library is pretty bare-bones and thus it would be less convenient (and more error-prone) to use it directly. |
In preparation for #5417, which I started looking into.