Skip to content
This repository has been archived by the owner on Nov 5, 2024. It is now read-only.

[Refactor] HomeViewModel: use by delegation for mutable states #3505

Closed
1 task done
ILIYANGERMANOV opened this issue Sep 14, 2024 · 4 comments · Fixed by #3517
Closed
1 task done

[Refactor] HomeViewModel: use by delegation for mutable states #3505

ILIYANGERMANOV opened this issue Sep 14, 2024 · 4 comments · Fixed by #3517
Assignees
Labels
approved Approved by the Ivy Wallet team. Ready for dev devexp Codebase & dev experience good first issue Good for newcomers keep Keep it from automatically getting closed by Stale

Comments

@ILIYANGERMANOV
Copy link
Collaborator

Please confirm the following

What would you like to improve?

Rework val x = mutableStateOf() and x.value to x by mutableStateOf() and x.

Because...

It's more convenient to use and the code becomes simpler to read.

Description

No response

Success Criteria

  • No functional changes
  • The app works as expected
@ILIYANGERMANOV ILIYANGERMANOV added devexp Codebase & dev experience good first issue Good for newcomers approved Approved by the Ivy Wallet team. Ready for dev keep Keep it from automatically getting closed by Stale labels Sep 14, 2024
@ivywallet
Copy link
Collaborator

Thank you @ILIYANGERMANOV for raising Issue #3505! 🚀
What's next? Read our Contribution Guidelines 📚.

Tagging @ILIYANGERMANOV for review & approval 👀

@ILIYANGERMANOV ILIYANGERMANOV changed the title [Refactor] HomeViewModel use by delegation for mutable states [Refactor] HomeViewModel: use by delegation for mutable states Sep 14, 2024
@prathmeshkc
Copy link
Contributor

I'm on it

@ivywallet
Copy link
Collaborator

Thank you for your interest @prathmeshkc! 🎉
Issue #3505 is assigned to you. You can work on it! ✅

If you don't want to work on it now, please un-assign yourself so other contributors can take it.

Also, make sure to read our Contribution Guidelines.

@ILIYANGERMANOV
Copy link
Collaborator Author

@shamim-emon and everyone one else interested, please feel free to create more issues like this one for the rest of the VMs

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
approved Approved by the Ivy Wallet team. Ready for dev devexp Codebase & dev experience good first issue Good for newcomers keep Keep it from automatically getting closed by Stale
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants