LemmyNet / jerboa

A native android app for Lemmy
GNU Affero General Public License v3.0
1.16k stars 166 forks source link

Fix migration changing the the app settings #1442

Closed MV-GH closed 6 months ago

MV-GH commented 6 months ago

This is really annoying, the order of these column definitions is very important.

This caused half the settings to be switched with two columns lower.

This fixes that

It is actually miracle that this didnt cause any crashes (well if it did we would have found it earlier). I don't see how to add a test for this to prevent this from happening again.

We will have to be very careful with doing big changes to appSettings again

Fixes #1440

I have verified this with a pre 30 db version app and 31 has the exact same settings.