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

Initialize Rolodex #2569

Merged
merged 3 commits into from
Jan 29, 2025
Merged

Initialize Rolodex #2569

merged 3 commits into from
Jan 29, 2025

Conversation

MitchellGoodwin
Copy link
Contributor

Creates the start of the Rolodex app.

Sets up the app with the initial routes setup. Since the native Contacts app does not open on its home screen, and instead opens on the default list of contacts, it felt important to set that up first. Some basic app state management is also setup for managing contact lists across the app.

Boilerplate visuals are setup for two of the screens. Improved fidelity will come later.

Screenshot 2025-01-27 at 12 09 57 PM Screenshot 2025-01-27 at 12 04 07 PM

Copy link

@dkwingsmt dkwingsmt left a comment

Choose a reason for hiding this comment

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

LGTM

@MitchellGoodwin
Copy link
Contributor Author

Please add this app to the CI configuration in:

Thanks!

Added to the master file, but the idea was to only test against master while we are getting this setup. Work on this is going on in parallel in the framework.

@ericwindmill
Copy link
Contributor

Please add this app to the CI configuration in:

Thanks!

Added to the master file, but the idea was to only test against master while we are getting this setup. Work on this is going on in parallel in the framework.

Thats fine. The CI failures are due to another sample, so feel free to land this.

@MitchellGoodwin MitchellGoodwin merged commit d62c784 into flutter:main Jan 29, 2025
7 of 13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants