Skip to content
This repository was archived by the owner on Aug 22, 2025. It is now read-only.

Conversation

@X1nto
Copy link
Member

@X1nto X1nto commented Apr 14, 2023

  • Separate stateless and stateful composables
  • Move ViewModel controlling from HomeScreen to individual panels
    • This removes the need for HomeScreen to call load() functions when data changes. Instead, ViewModels observe data changes themselves and update the UI accordingly. This should ensure that all data is updated properly.
  • Refactor navigation
  • Get rid of BasePersistenceViewModel
  • Refactor smaller components, such as messages, etc.

viewModelScope.launch {
val guild = guilds.fetchGuild(persistentGuildId)
?: return@launch
persistentDataStore.observeCurrentGuild()
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this does not need to be updated when the current guild changes (it won't except for a tiny edge case)

it should stay the same for as long as the screen is open

should probably add some getGuildId() or something to persistent store

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I can just make it get the last value, no problems with that

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants