We will eventually end up using some library for making the app network proof
I don’t wanna get into this currently because the basic norm is to first make an internet always only app and then implement caching and local stuff for network calls
The biggest example is that even the websites just die down as soon as the internet is gone
jasje
um so no cache
?
akshaaatt
Yes to cache but not right now with the current bandwidth we have
I'm fine with it if you wanna explore stuff and send me more libraries
jasje
welp i was just looking to cache the reviews a user would write that would be drafted, but now then i went towards total caching
but then**
akshaaatt
That sounds good to me jasje
For that use case of yours, roomdb would be good
Basically drafts
jasje
I was thinking of that too
but i thought maybe something more automated?
akshaaatt
Btw, I need some help with testing a few things jasje. Can you please pull the latest main branch and try to play with it? I have revamped a lot of stuff. Also, I need some help with switching the theme. Idk what the issue is there.
Gimme a min before I merge
jasje
alrite
woah 1.3k
a lot of work you got in
akshaaatt
Yeah it’s as much as 1/4th of a gsoc project lol
jasje
lol
i thought rip profile
but yay
also about the removal of settings
is this the right step?
akshaaatt
Nothing is rip here. We need to complete this app in terms of features by the end of this year
Yes jasje. We need a better UX. Just giving a settings panel is not a good ux
Also why tf do we even need settings for? Our app doesn’t even have any settings as such
jasje
we could have later on tho
akshaaatt
This will also give us the right motivation to move to compose completely
jasje
just look at website
akshaaatt
Yeah, we’ll it in JC then
jasje
that i agree it
agree with**
so about caching
akshaaatt
Tbh I’m bot a fan of settings even kn the website
Not*
jasje
me fan
akshaaatt
Lol
Everything should just be jn the basic profile page itself tbh
jasje
its intuitive to look into settings for something "core"
akshaaatt
I agree with it but there shouldn’t be a separate settings panel for the app until and unless your app has only one functionality
jasje
we can remove it for now
akshaaatt
For example, we want to give the user to control their bass, etc for BP on the app
Where would those settings be?
jasje
+1
could have it in bp too
akshaaatt
Exactly
jasje
I think we are drifting from the focus here
caching
akshaaatt
Lol
I did change the topic so we didn’t drift actually
[listenbrainz-android] 14dependabot[bot] closed pull request #151 (03main…dependabot/gradle/com.google.android.material-material-1.9.0): Bump com.google.android.material:material from 1.8.0 to 1.9.0 https://github.com/metabrainz/listenbrainz-andr...
[listenbrainz-android] 14dependabot[bot] closed pull request #149 (03main…dependabot/gradle/compose_version-1.4.3): Bump compose_version from 1.4.2 to 1.4.3 https://github.com/metabrainz/listenbrainz-andr...
[listenbrainz-android] 14dependabot[bot] closed pull request #153 (03main…dependabot/gradle/org.jetbrains.kotlinx-kotlinx-coroutines-test-1.7.1): Bump org.jetbrains.kotlinx:kotlinx-coroutines-test from 1.6.4 to 1.7.1 https://github.com/metabrainz/listenbrainz-andr...
akshaaatt
Yeah we should definitely improve BP.
BrainzGit
[listenbrainz-android] 14dependabot[bot] closed pull request #152 (03main…dependabot/gradle/app.cash.turbine-turbine-0.13.0): Bump app.cash.turbine:turbine from 0.12.3 to 0.13.0 https://github.com/metabrainz/listenbrainz-andr...
jasje
currently we do kotlin calcs or runtime calcs which is slower
akshaaatt
Yus
jasje
also keep in mind to use viewpager for loading songs in bp screen
akshaaatt
I’m happy with it if you wanna update those jasje
jasje
akshaaatt: we could have a ticket
a lot of stuff needs updating
akshaaatt
Sure
jasje
priorities are must
akshaaatt
Yeah you’ll soon realize that everything needs to be maintained and updated
In a while, you’ll realize the need to refactor YIM stuff as well
But again, as even you said, priorities will matter then
jasje
this app needs a fulltime dev
no cap
upgraded gradle version huh
akshaaatt
Lol
jasje
akshaaatt: not sure about this one
the some bottom nav icons are different in terms of shape and size when selected
while others arent
about the system theme
we were using isUiModeDark for that
when you remove the activity restart thing it works fine
we arent updating the preference here.
akshaaatt
Ouu right
jasje
too much done in this pr
akshaaatt
Can you please make a PR for it? Also, about the icons, I couldn’t find the same icons for free on fontawesome. I need but selected and unselected. After beta release, we can get aerozol to provide us the icons
Both*
jasje
let me see
akshaaatt has been working since morning today and is now headed out with friends for a drink or two
akshaaatt: where did you get these theme icons
its confusing to use two
separate icons
plus both have black tints
🫠
akshaaatt
Fontawesome, jasje
jasje
wait
we dont have white tints at all?
akshaaatt
Reddit uses this as well
Moon and sun to switch the theme
Ouu they don’t use sun
Fine, let’s do one icon then
jasje
akshaaatt: can you please do "validate token" call as well?