mopsalarm / Pr0

Offizielle pr0gramm app für Android. Download auf https://app.pr0gramm.com
MIT License
284 stars 39 forks source link

Use different endpoint for Seen-Sync #268

Closed holzmaster closed 1 year ago

holzmaster commented 1 year ago

Der Endpunkt ist aktuell nur für Mods zugreifbar, d.h. das Mergen würde das aktuell für normale User kaputt machen.

Ich werde das mal bei mir selbst kompilieren und ein bisschen Dogfooden. Im Emulator hat's bei mir funktioniert. Falls du (@mopsalarm) lust hast, kannst du es ja auch mal testen und/oder ggf. schon reviewen.

Wirklich dringend ist der PR nicht.

Dann bleibt noch eine Frage: Der KV-Service wird hiernach nicht mehr verwendet, weil's nichts anderes gibt, was in dem KV-Store gespeichert wird. Soll weg oder lieber drinne lassen?

mopsalarm commented 1 year ago

Ja, mach mal den KVStore dann weg.

holzmaster commented 1 year ago

Mir ist noch aufgefallen: Ist die lokal gespeicherte Datei (seen-posts.bits) userspezifisch? Wird die gelöscht, wenn man sich ausloggt bzw. bevor man sich mit einem anderen Account einloggt? Ich habs jetzt nicht genauer gedebuggt bzw. nachgeschaut, ist nur eine Frage, ich ich mir gerade stelle.

mopsalarm commented 1 year ago

Die lokale seen-posts.bits wird beim logout geleert.

mopsalarm commented 1 year ago

Wann willst du das denn live nehmen?

holzmaster commented 1 year ago

Ist generell nicht dringend und hat noch ein paar Wochen bis Monate. Mir ist da lieber, dass das ordentlich getestet ist. Wir können es ja schon in einen anderen Zielbranch mergen.

mopsalarm commented 1 year ago

Ne, dann brauchst du es auch nicht mergen. In einem anderen branch ist es jetzt ja auch schon. Testen läuft bei mir meistens so, dass ich das, wenns bei mir lokal läuft, als Beta an ein paar Prozent der User ausrolle und dann eine Weile beobachte. Naja, sag einfach bescheid, wenn du meinst, dass alles läuft.

holzmaster commented 1 year ago

Ich hab die eine Sache noch geändert. Passt das so?

mopsalarm commented 1 year ago

Sorry, bin gerade im Urlaub. Ich gucke gerne am Montag genauer drauf.

holzmaster commented 1 year ago

Wie läuft das, wenn wir das ausrollen, unsere internen Tests funktionieren und die Beta gut läuft? Kann man so ein Update auch für alle Nutzer beim App-Start forcieren? Wie konservativ waren wir da bisher mit EOL-Versionen? Wär ggf. interessant dafür, um die Nutzung des Endpunkts möglichst schnell hochzudrehen.

mopsalarm commented 1 year ago

Okay, draufgucken habe ich noch nicht geschafft, aber morgen denke ich.

Ja, man kann ein Update forcieren, würde ich aber frühestens nach zwei oder drei Wochen "stable" machen, sonst gibt's wieder Mimimi.

holzmaster commented 1 year ago

Ja, man kann ein Update forcieren, würde ich aber frühestens nach zwei oder drei Wochen "stable" machen, sonst gibt's wieder Mimimi.

Hört sich super an. :)

mopsalarm commented 1 year ago

Schaltest du den Endpunkt für alle frei? Dann merge ich das nachher und teste das morgen nochmal, bevor ich ne Beta Version baue. Heute habe ich leider keine Zeit.

holzmaster commented 1 year ago

Der Endpunkt ist schon für alle freigeschaltet; kannst schon testen. Ich verwende seit ein paar Wochen auch schon einen Debug-Build, der diesen PR verwendet.

mopsalarm commented 1 year ago

Morgen hat sich etwas hingezogen. Ich bau gerade die Beta und schiebe rolle sie dann an ein paar Prozent der Nutzer aus.

mopsalarm commented 1 year ago

Wie läufts serverseitig?

holzmaster commented 1 year ago

Da steht alles soweit, auch für alle Nutzer. Geht etwas nicht? Ich verwende bei mir abwechselnd meinen Debug-Build und die App mit Beta-Channel. Da geht alles soweit.

holzmaster commented 1 year ago

Ist das Update schon für alle User live?

mopsalarm commented 1 year ago

Nope, aber dann rolle ich das jetzt an alle aus.