search

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 #46

Closed Zusel closed 9 months ago

Zusel commented 11 months ago

Heyho,

ursprünglich wollte ich nur das Feature mit dem "Gelesen"-Status hinzufügen. Da ist wohl etwas außer Kontrolle geraten ^^ Ich hoffe, das passt trotzdem so. Wenn ihr noch Anmerkungen habt, immer her damit. Ich beziehe mich auf das Issue #11

LG

Zusel commented 9 months ago

Ping @eknoes 🙃

eknoes commented 9 months ago

Sorry, das war nicht in Ordnung dass ich das solange liegen lasse habe. Reviewe + teste beide jetzt einmal ordentilch, um sie dann zu veröffentlichen.

Zusel commented 9 months ago

Heyho, alles gut. Ich hatte tatsächlich gestern auch Mal kurz drüber nachgedacht nen orm-mapper einzubauen. Damit hätte man die Probleme nicht mehr. Ich würde mich da heute Mal hinsetzen. Was ist deine Meinung zum Thema ORM-Mapper?

eknoes commented 9 months ago

Ich habe da keine Meinung zu :D Wenn du da Lust zu hast gerne. Ich habe die App vor Jahren gebaut um ein bisschen Android Entwicklung zu lernen. Scheint so, als ob Google das für Android mittlerweile auch so empfiehlt: https://developer.android.com/training/data-storage/room

Gut fände ich, wenn die DB dann auch migriert wird, anstatt sie zu löschen und neu anzulegen, damit die ganzen alten Artikel noch drin bleiben die nicht mehr im Feed sind.

Das einzige was mir wirklich wichtig ist, ist dass keine Cloud/Tracking Features rein kommen. Ansonsten tob dich gerne aus :) Ich werd in Zukunft auch schneller testen und dann veröffentlichen!

Zusel commented 9 months ago

Also kann dieser PR erstmal als obsolet angesehen werden und ich schaue mir das mit dem ORM-Mapper erst an

eknoes commented 9 months ago

Wenn du da Lust zu hast gerne. Danke! :)