Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rewrite AccountSettingsActivity to M3 #795

Merged
merged 11 commits into from
May 16, 2024

Conversation

sunkup
Copy link
Member

@sunkup sunkup commented May 16, 2024

Purpose

More of the UI is becoming M3 ✨

Short description

  • Extract composable
  • Extract model
  • Switch to M3

Checklist

  • The PR has a proper title, description and label.
  • I have self-reviewed the PR.
  • I have added documentation to complex functions and functions that can be used by other modules.
  • I have added reasonable tests or consciously decided to not add tests.

@sunkup sunkup force-pushed the rewrite-accountsettingsactivity-to-m3 branch from 29bf3f7 to c150ec9 Compare May 16, 2024 13:03
@sunkup sunkup changed the title Rewrite AccountSettingsActivity to mḾ3 Rewrite AccountSettingsActivity to M3 May 16, 2024
@sunkup sunkup marked this pull request as ready for review May 16, 2024 13:13
@sunkup sunkup requested a review from rfc2822 May 16, 2024 13:14
@rfc2822 rfc2822 force-pushed the rewrite-accountsettingsactivity-to-m3 branch from 7f6f416 to de1cb71 Compare May 16, 2024 15:06
@rfc2822 rfc2822 force-pushed the rewrite-accountsettingsactivity-to-m3 branch from 402658a to c9528d5 Compare May 16, 2024 15:29
@rfc2822 rfc2822 added the refactoring Internal improvement of existing functions label May 16, 2024
@rfc2822 rfc2822 merged commit e2bfa8c into main-ose May 16, 2024
6 checks passed
@rfc2822 rfc2822 deleted the rewrite-accountsettingsactivity-to-m3 branch May 16, 2024 15:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactoring Internal improvement of existing functions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants