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

Investigate using finish instead of intent #3641

Closed
2 tasks
Sparsh1212 opened this issue Aug 4, 2021 · 4 comments · Fixed by #5557
Closed
2 tasks

Investigate using finish instead of intent #3641

Sparsh1212 opened this issue Aug 4, 2021 · 4 comments · Fixed by #5557
Assignees
@theayushyadav11
Labels
enhancement End user-perceivable enhancements. good first issue This item is good for new contributors to make their pull request. Impact: Low Low perceived user impact (e.g. edge cases). Issue: Needs Clarification Indicates that an issue needs more detail in order to be able to be acted upon. Work: Low Solution is clear and broken into good-first-issue-sized chunks. Z-ibt Temporary label for Ben to keep track of issues he's triaged.

Comments

@Sparsh1212
Copy link
Contributor

Sparsh1212 commented Aug 4, 2021 •
edited by adhiamboperes
Loading

Currently, we need to start intent for ProfileChooserActivity to get an update. We need to investigate on using finish instead as the live data bug is fixed.
@Broppia Broppia added issue_type_infrastructure Impact: Low Low perceived user impact (e.g. edge cases). labels Jul 7, 2022
@BenHenning BenHenning added Issue: Needs Clarification Indicates that an issue needs more detail in order to be able to be acted upon. Z-ibt Temporary label for Ben to keep track of issues he's triaged. issue_user_developer labels Sep 15, 2022
@seanlip seanlip added bug End user-perceivable behaviors which are not desirable. and removed issue_type_infrastructure labels Mar 28, 2023
@seanlip seanlip added enhancement End user-perceivable enhancements. and removed bug End user-perceivable behaviors which are not desirable. Type: Improvement labels Jun 16, 2023
@adhiamboperes adhiamboperes added the Work: Medium The means to find the solution is clear, but it isn't at good-first-issue level yet. label Aug 10, 2023
@adhiamboperes adhiamboperes added the good first issue This item is good for new contributors to make their pull request. label Oct 28, 2023
@rishav2404
Copy link

I want to work on this issue. Please assign this to me.

@adhiamboperes
Copy link
Collaborator

I want to work on this issue. Please assign this to me.

Hi @rishav2404, please give an overview of your proposed solution

@adhiamboperes adhiamboperes added Work: Low Solution is clear and broken into good-first-issue-sized chunks. and removed Work: Medium The means to find the solution is clear, but it isn't at good-first-issue level yet. labels May 17, 2024
@adhiamboperes adhiamboperes mentioned this issue Jul 17, 2024
Open
6 tasks
@theayushyadav11
Copy link
Contributor

I want to work on this issue,plz assign it to me.

@adhiamboperes
Copy link
Collaborator

@theayushyadav11, please feel free to create a PR.

theayushyadav11 added a commit to theayushyadav11/oppia-android that referenced this issue Oct 21, 2024
@theayushyadav11
Fix: Use finish() instead of intent for smoother navigation (oppia#3641)
54c7bd1
@theayushyadav11 theayushyadav11 mentioned this issue Oct 21, 2024
Merged
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@theayushyadav11 theayushyadav11

Labels
enhancement End user-perceivable enhancements. good first issue This item is good for new contributors to make their pull request. Impact: Low Low perceived user impact (e.g. edge cases). Issue: Needs Clarification Indicates that an issue needs more detail in order to be able to be acted upon. Work: Low Solution is clear and broken into good-first-issue-sized chunks. Z-ibt Temporary label for Ben to keep track of issues he's triaged.
Projects
[Team] Core Learner and Mastery flows & UI Frontend - Android
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fixes #3641 Use finish() instead of intent for smoother navigation theayushyadav11/oppia-android
7 participants
@seanlip @BenHenning @Sparsh1212 @adhiamboperes @Broppia @rishav2404 @theayushyadav11