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.

Sign up for GitHub

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

Jump to bottom

Fixed #10 issue. #14

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

Fixed #10 issue. #14

wants to merge 10 commits into from

Conversation

jinosh05
Copy link

@jinosh05 jinosh05 commented Sep 19, 2022 •
edited
Loading

I have upgraded some minor packages and fixed the null check by adding an if condition and another condition to make other values not empty. Here #10 and #3 are fixed.
Thank you

@jinosh05
Copy link
Author

Hello there, I have made the User value nullable . Can you give a try and accept pull request

@jinosh05
Copy link
Author

jinosh05 commented Dec 4, 2022

Test my recent commit once

Ultra-rrich
Ultra-rrich reviewed Mar 16, 2023
View reviewed changes
Copy link

@Ultra-rrich Ultra-rrich left a comment

Choose a reason for hiding this comment

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

Hey friend, so I tried your solution looks like it only cleared the error screen.
However, the after selecting or taking a photo, nothing happens. no state changes. Thhe upload button button is still stagnant

@Aishwarya2701 Aishwarya2701 mentioned this pull request Oct 1, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Ultra-rrich Ultra-rrich Ultra-rrich left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jinosh05 @Ultra-rrich