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

[$250] [Navigation Bug]: The Back button in the confirm page close RHP #47697

Closed
dylanexpensify opened this issue Aug 20, 2024 · 33 comments
Closed
Assignees
Labels
External Added to denote the issue can be worked on by a contributor Monthly KSv2 Reviewing Has a PR in review Task

Comments

@dylanexpensify
Copy link
Contributor

dylanexpensify commented Aug 20, 2024

Coming from this project, we have this follow-up polish issue we need to complete where the back button in the confirm page closes the RHP. More details here.

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~0134a56cd991934297
  • Upwork Job ID: 1825862477363321355
  • Last Price Increase: 2024-08-20
Issue OwnerCurrent Issue Owner: @
@dylanexpensify
Copy link
Contributor Author

cc @blazejkustra

@blazejkustra
Copy link
Contributor

Hey! I’m Błażej Kustra from Software Mansion, an expert agency, and I’d like to work on this issue!

@dylanexpensify dylanexpensify added the External Added to denote the issue can be worked on by a contributor label Aug 20, 2024
Copy link

melvin-bot bot commented Aug 20, 2024

Job added to Upwork: https://www.upwork.com/jobs/~0134a56cd991934297

@melvin-bot melvin-bot bot changed the title [Navigation Bug]: The Back button in the confirm page close RHP [$250] [Navigation Bug]: The Back button in the confirm page close RHP Aug 20, 2024
@melvin-bot melvin-bot bot added the Help Wanted Apply this label when an issue is open to proposals by contributors label Aug 20, 2024
Copy link

melvin-bot bot commented Aug 20, 2024

Current assignee @DylanDylann is eligible for the External assigner, not assigning anyone new.

@dylanexpensify
Copy link
Contributor Author

Hi @blazejkustra! It's all yours!

@blazejkustra
Copy link
Contributor

Before:

Bug: The Back button in the confirm page should navigate back to approver selection page (link)

356328274-2b92fcaa-8970-4b16-a10f-42e20d417689.mov

After:
Going back on create approval screen navigates back to the first approver screen as shown on the video.

Screen.Recording.2024-08-20.at.15.46.46.mov

Before I proceed with implementation I'd like to ask about your input @JmillsExpensify @tgolen @Expensify/design - do you prefer this updated approach?

@Lalarukh-khan
Copy link

I have also applied on Upwork. Can I get an opportunity to get it done?

Copy link

melvin-bot bot commented Aug 20, 2024

📣 @Lalarukh-khan! 📣
Hey, it seems we don’t have your contributor details yet! You'll only have to do this once, and this is how we'll hire you on Upwork.
Please follow these steps:

  1. Make sure you've read and understood the contributing guidelines.
  2. Get the email address used to login to your Expensify account. If you don't already have an Expensify account, create one here. If you have multiple accounts (e.g. one for testing), please use your main account email.
  3. Get the link to your Upwork profile. It's necessary because we only pay via Upwork. You can access it by logging in, and then clicking on your name. It'll look like this. If you don't already have an account, sign up for one here.
  4. Copy the format below and paste it in a comment on this issue. Replace the placeholder text with your actual details.
    Screen Shot 2022-11-16 at 4 42 54 PM
    Format:
Contributor details
Your Expensify account email: <REPLACE EMAIL HERE>
Upwork Profile Link: <REPLACE LINK HERE>

@Lalarukh-khan
Copy link

Contributor details
Your Expensify account email: [email protected]
Upwork Profile Link: https://www.upwork.com/freelancers/~01152db88584b3187c

Copy link

melvin-bot bot commented Aug 20, 2024

✅ Contributor details stored successfully. Thank you for contributing to Expensify!

@belawalumer
Copy link

Contributor details
Your Expensify account email: [email protected]
Upwork Profile Link: https://www.upwork.com/freelancers/~01f81b79aedaff801c

Copy link

melvin-bot bot commented Aug 20, 2024

✅ Contributor details stored successfully. Thank you for contributing to Expensify!

@kanha55
Copy link

kanha55 commented Aug 20, 2024

hello
Where I can submit the proposals on GitHub?

Copy link

melvin-bot bot commented Aug 20, 2024

📣 @kanha55! 📣
Hey, it seems we don’t have your contributor details yet! You'll only have to do this once, and this is how we'll hire you on Upwork.
Please follow these steps:

  1. Make sure you've read and understood the contributing guidelines.
  2. Get the email address used to login to your Expensify account. If you don't already have an Expensify account, create one here. If you have multiple accounts (e.g. one for testing), please use your main account email.
  3. Get the link to your Upwork profile. It's necessary because we only pay via Upwork. You can access it by logging in, and then clicking on your name. It'll look like this. If you don't already have an account, sign up for one here.
  4. Copy the format below and paste it in a comment on this issue. Replace the placeholder text with your actual details.
    Screen Shot 2022-11-16 at 4 42 54 PM
    Format:
Contributor details
Your Expensify account email: <REPLACE EMAIL HERE>
Upwork Profile Link: <REPLACE LINK HERE>

@kanha55
Copy link

kanha55 commented Aug 20, 2024

Contributor details
Your Expensify account email: [email protected]
Upwork Profile Link: https://www.upwork.com/freelancers/~0125c7c7adee611508

Copy link

melvin-bot bot commented Aug 20, 2024

✅ Contributor details stored successfully. Thank you for contributing to Expensify!

@kanha55
Copy link

kanha55 commented Aug 20, 2024

I am submitting my proposal
Step 1: Understand the Issue

The back button on the confirm page should navigate to the previous page but instead closes the Right-Hand Panel (RHP).

Step 2: Check the Code

Identify the specific component responsible for the back button functionality within the confirm page.
Analyze the navigation logic, focusing on the current behavior of the back button.

Step 3: Review Approvers Page

Investigate how the navigation works on the Approvers page (RHP) since it's working correctly.
Apply the same logic or implementation to the confirm page (RHP).

Step 4: Fix the Navigation on confirm page

Modify the back button's action on the confirm page to ensure it correctly navigates to the approver selection page rather than closing the RHP.

Step 5: Test the Fix

Test the functionality by navigating through the pages to confirm that the issue is resolved and the back button behaves as expected.

Step 6: Write Test Cases

Write the necessary test cases to cover the changes made to the back button functionality.

Step 7: Create a Pull Request

Once the fix is confirmed, submit a pull request with a detailed explanation of the changes, including the logic behind the fix and the tests you've written.

@blazejkustra
Copy link
Contributor

@dylanexpensify Could you remove Help wanted tag? 😅

@dylanexpensify dylanexpensify removed the Help Wanted Apply this label when an issue is open to proposals by contributors label Aug 21, 2024
@JmillsExpensify
Copy link

@blazejkustra yes, we prefer the updated approach!

@melvin-bot melvin-bot bot added Reviewing Has a PR in review Weekly KSv2 and removed Daily KSv2 labels Aug 22, 2024
@blazejkustra
Copy link
Contributor

@DylanDylann PR is up!

Copy link

melvin-bot bot commented Aug 26, 2024

Triggered auto assignment to @tylerkaraszewski, see https://stackoverflow.com/c/expensify/questions/7972 for more details.

Copy link

melvin-bot bot commented Aug 27, 2024

⚠️ Looks like this issue was linked to a Deploy Blocker here

If you are the assigned CME please investigate whether the linked PR caused a regression and leave a comment with the results.

If a regression has occurred and you are the assigned CM follow the instructions here.

If this regression could have been avoided please consider also proposing a recommendation to the PR checklist so that we can avoid it in the future.

Copy link

melvin-bot bot commented Sep 16, 2024

This issue has not been updated in over 15 days. @tylerkaraszewski, @blazejkustra, @DylanDylann eroding to Monthly issue.

P.S. Is everyone reading this sure this is really a near-term priority? Be brave: if you disagree, go ahead and close it out. If someone disagrees, they'll reopen it, and if they don't: one less thing to do!

@melvin-bot melvin-bot bot added the Monthly KSv2 label Sep 16, 2024
@DylanDylann
Copy link
Contributor

@tylerkaraszewski Please help process the payment for me. Thanks

@tylerkaraszewski
Copy link
Contributor

Not sure why @dylanexpensify was unassigned, he should be able to do that.

@dylanexpensify
Copy link
Contributor Author

Payment summary:

Contributor+: @DylanDylann $250 via Upwork

Please apply/request!

@DylanDylann
Copy link
Contributor

DylanDylann commented Oct 5, 2024

@dylanexpensify The Upwork job is no longer available, could you create a new job and send the offer to my profile here? https://www.upwork.com/freelancers/~010c6310ab55f3d17d

Thanks

@dylanexpensify
Copy link
Contributor Author

@DylanDylann yes!

@dylanexpensify
Copy link
Contributor Author

@DylanDylann sent invite!

@DylanDylann
Copy link
Contributor

@dylanexpensify thanks, I applied to the job

@dylanexpensify
Copy link
Contributor Author

@DylanDylann sent!

@DylanDylann
Copy link
Contributor

@dylanexpensify accepted!

@dylanexpensify
Copy link
Contributor Author

Done!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
External Added to denote the issue can be worked on by a contributor Monthly KSv2 Reviewing Has a PR in review Task
Projects
Archived in project
Development

No branches or pull requests

8 participants