search

ICT4Dat / ict4dat-news-android

ICT4D.at's App to combine all ICT4D news into one Android application
Apache License 2.0
5 stars 1 forks source link

Fixes #123 - News read state #143

Closed rajasone closed 4 years ago

spipau commented 5 years ago

Hi,

can you explain me quickly your steps? Why did you create a new Model? I would have just added a new variable

@ColumnInfo(name = NEWS_TABLE_READ)
val read: Boolean = false

to the News class. Then the background changes depending on this value in the list and when a list item is pressed, then the variable changes to true and gets stored to the DB. Isn't that much easier? Please let me know what you think!

(haha, sorry for destroying your hard work! I hope you don't get me wrong. I'm sure your solutions also work very well and it's amazing what you can already do (!), but sometimes you over-complicate things 😉 )

rajasone commented 5 years ago

@spipau yes you are absolutely right let me implement it in easy way, really sorry for over-complected solution