eknoes / golem-android-reader

An unofficial android application for german tech-site golem.de
https://play.google.com/store/apps/details?id=de.eknoes.inofficialgolem
GNU General Public License v3.0
20 stars 6 forks source link

Show already read and DB-Stuff #48

Open Zusel opened 7 months ago

Zusel commented 7 months ago

Heyho, ich glaube der PR sollte inzwischen bekannt sein ^^ Ist noch nicht ganz fertig da ich Performance Probleme mit dem Main-Thread habe. Nebenbei räume ich auch noch nen paar Sachen auf. Nen Version-Bumper kommt dann in nem separaten PR. LG Flo

eknoes commented 7 months ago

Cool, danke! Gibts die Möglichkeit, die alten DB Inhalte in die neue zu überführen? Oder wird bei einem Upgrade alles gelöscht und es geht von vorne los?

Zusel commented 7 months ago

Cool, danke! Gibts die Möglichkeit, die alten DB Inhalte in die neue zu überführen? Oder wird bei einem Upgrade alles gelöscht und es geht von vorne los?

Geht bestimmt. Die Frage ist ob der Mehrwert da jetzt so gegeben ist. Man müsste ehern schauen wie man die gelöscht bekommt ^^. Ich habe aber noch nen UI-Problem, was ich noch nicht gelöst bekommen habe. Da das ganze Room-Zeug nicht im Main-Thread läuft, muss ich das irgendwie synchron bekommen. Da hänge ich derzeit noch etwas.

eknoes commented 7 months ago

Die Android Docs haben anscheinend zu dem asynchronen was (https://developer.android.com/training/data-storage/room/async-queries) als auch zu der Migration: https://developer.android.com/training/data-storage/room/sqlite-room-migration#incremental

Ich glaube einen riesigen Mehrwert gibt es nicht. Ich kann mir vorstellen, dass einzelne User da mittlerweile echt eine Menge Artikel drin haben, der RSS-Feed gibt ja nur die aktuellsten 20 (?) oder so. Ich hatte mal ganz früher den Plan, eine lokale Suche einzubauen, dafür wäre das relevant. Aber wenn das ein größerer Akt ist mit der Migration ist es auch okay das zu lassen, denke ich!

Zusel commented 7 months ago

Ah, das zur Migration kannte ich noch nicht. Hatte ich aber auch noch nicht drüber nachgedacht. Ja es gibt einige Seiten die beschreiben, wie man das mit dem async am besten macht. Ist aber in meinem jetzigen Plan, mit sehr viel Umbauten verbunden. Das hab ich leider noch nicht geschafft. Da mir da auch noch die Erfahrung fehlt, tuh ich mich da etwas schwer. Aber bekomme ich schon hin^^

Zusel commented 6 months ago

Hey, nur ne kleine Info. Ich bin gerade wieder aus'm Ausland gekommen und habe jetzt noch nen paar Projekte aufm Tisch. Ich hab's nicht vergessen und werde es auch noch umsetzen. Leider wird das noch ne Weile dauern. Sorry dafür.

eknoes commented 6 months ago

Gar kein Problem. Ohne dich würde hier gar nichts passieren, also mach dir darum keinen Kopf :)

Zusel commented 4 months ago

Sooooooooo, was lange währt wird gut ^^ Also hier mal der PR. Vorweg: Ich habe ausversehen den Branch auf mein Main gerebased (da ist der GH-Workflow mit build drinnen). Deswegen bitte erst den BuildAPK PR mergen. Sowieso muss der erstmal getestet werden. Ich werde mir die APK hier auch mal auf mein Handy ziehen und schauen ob das Probleme macht. Da ich sehr viel geändert habe, bin ich mir hier nicht sicher ob das alles so gut ist. Also bitte teste das auch mit..... Ich würde mal so vorschlagen das wir in ca 2-3 Wochen nochmal drüber quatschen/schreiben.

Zusel commented 4 months ago

Feedback: das gelesen Feature funktioniert irgendwie nicht nicht richtig. Kann noch nicht wirklich nachvollziehen wann und wieso das noch nicht klappt.

Beschreibung: wird nicht als gelesen angezeigt obwohl ich den Artikel angeklickt habe

Zusel commented 4 months ago

To-Do: Migration schreiben

Zusel commented 4 months ago

Migration ist fertig

eknoes commented 3 months ago

Danke! Dann teste ich das mal ein bisschen. Vor dem normalen Rollout würde ich das auch erstmal im Play Store auf dem Beta Track releasen.

Würde die Changes nochmal in Ruhe zu einem anderen Zeitpunkt ansehen.

Feedback: das gelesen Feature funktioniert irgendwie nicht nicht richtig. Kann noch nicht wirklich nachvollziehen wann und wieso das noch nicht klappt.

Beschreibung: wird nicht als gelesen angezeigt obwohl ich den Artikel angeklickt habe

Ist das noch der Fall?

Zusel commented 3 months ago

Feedback: das gelesen Feature funktioniert irgendwie nicht nicht richtig. Kann noch nicht wirklich nachvollziehen wann und wieso das noch nicht klappt. Beschreibung: wird nicht als gelesen angezeigt obwohl ich den Artikel angeklickt habe

Ist das noch der Fall?

Ja, ich hab gestern nur noch die Migration geschrieben aber noch nicht geschaut was er denn für nen Problem mit dem gelesen Feature hat.