Closed kbecciv closed 1 year ago
Triggered auto assignment to @adelekennedy (Bug
), see https://stackoverflow.com/c/expensify/questions/14418 for more details.
Platforms
in OP are ✅)When click back button, app displays the same welcome message page This happens on mac/chrome as well
ReportWelcomeMessagePage
and ReportSettingsPage
are in different stack navigator
I think the root cause is same as here
We can replace the current path instead of going back.
with
<HeaderWithBackButton
title={props.translate('welcomeMessagePage.welcomeMessage')}
onBackButtonPress={() => Navigation.navigate(ROUTES.getReportSettingsRoute(props.report.reportID, 'UP'))}
/>
updateWelcomeMessage
function in Report.js, replace Navigation.goBack()
with Navigation.navigate(ROUTES.getReportSettingsRoute(reportID, 'UP'))
This works as expected.
Looks like something related to react-navigation
may have been mentioned in this issue discussion.
As a reminder, please make sure that all proposals are not workarounds and that any and all attempt to fix the issue holistically have been made before proceeding with a solution. Proposals to change our DeprecatedCustomActions.js
files should not be accepted.
Feel free to drop a note in #expensify-open-source with any questions.
@adelekennedy Whoops! This issue is 2 days overdue. Let's get this updated quick!
Job added to Upwork: https://www.upwork.com/jobs/~01974ff1b6dd73c766
Current assignee @adelekennedy is eligible for the External assigner, not assigning anyone new.
Triggered auto assignment to Contributor-plus team member for initial proposal review - @parasharrajat (External
)
little glitchy, good to fix
@parasharrajat, @adelekennedy Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!
one proposal to review - not doubling yet!
@s-alves10 Proposal missing a proper explanation of the root cause and also an explanation of how you are solving the root cause (not what code is doing but theory).
Waiting for proposals.
@parasharrajat
Please read this https://github.com/react-navigation/react-navigation/issues/6434
Can you still reproduce ? I think a change has been made in linkTo and that should solve it: https://github.com/Expensify/App/blob/main/contributingGuides/NAVIGATION.md#why-we-need-to-use-minimal-action-in-the-linkto-function
@s-alves10 I will go through that when possible but it is better if you can explain here.
I'm not able to reproduce (but I find the android reproduction steps tricky)
yeah. it's not reproducible now
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸
I'm going to close this one as I think this has been fixed - I think 25% is still due here for the proposal reviews
Hi @adelekennedy, @parasharrajat is it eligible for reporting bonus?
Ah! Yes - thank you for catching that @dhanashree-sawant will you apply here ?
@adelekennedy Requested 25% on NewDot.
Approved $250 based on https://github.com/Expensify/App/issues/22082#issuecomment-1636284267
Hi @adelekennedy, @anmurali , sorry I missed that message, if possible, can you send me invite to the job?
@dhanashree-sawant hired! Please accept and I'll pay out
Thanks @adelekennedy, Offer accepted.
Hi @adelekennedy, ping for approving the offer on upwork
If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!
Action Performed:
Expected Result:
App should display room settings when we click on back on welcome message page
Actual Result:
App displays same welcome message page when we save welcome message, reopen it and click on back
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Version Number: 1.3.35-5 Reproducible in staging?: y Reproducible in production?: y If this was caught during regression testing, add the test name, ID and link from TestRail: Email or phone of affected tester (no customers): Logs: https://stackoverflow.com/c/expensify/questions/4856 Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/36e02196-eaeb-4977-90d5-55b046eda68c
https://github.com/Expensify/App/assets/93399543/12fa4e3f-2662-47f7-88fe-90bb4c766bed
Expensify/Expensify Issue URL: Issue reported by: GH handle - @dhanashree-sawant Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1688225568421009
View all open jobs on GitHub
Upwork Automation - Do Not Edit