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

Updated Bottom NavigationBar with NullSafety #526

Closed
wants to merge 1 commit into from

Conversation

rahul31124
Copy link
Contributor

This pull request addresses a critical issue where the app was crashing when a user attempted to access the profile section from the bottom navigation while not logged in.

Issue Fixed:
The app was previously crashing when a user clicked on the profile option in the bottom navigation, and the user was not logged in.

Changes Implemented:
Null Safety: Added null and blank checks for the username to ensure that the app only tries to navigate to the profile if the username is valid (not null or blank).

User Experience Improvement: If the user is not logged in and attempts to navigate to the profile, an AlertDialog will appear prompting the user to log in, instead of the app crashing. The dialog provides the option to either log in or cancel the action.

Before

Before.mp4

After

After.mp4

@rahul31124
Copy link
Contributor Author

@07jasjeet need your help

@rahul31124 rahul31124 closed this by deleting the head repository Jan 21, 2025
@07jasjeet
Copy link
Collaborator

Hey @rahul31124, you can reach out to the team by going through the communication section in readme. Just tag me there, my nick is "jasje".

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.

2 participants