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] Android - NetSuite - Option is not being selected without scrolling first on a long list #44927

Closed
1 of 6 tasks
lanitochka17 opened this issue Jul 5, 2024 · 30 comments
Closed
1 of 6 tasks
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 Engineering External Added to denote the issue can be worked on by a contributor

Comments

@lanitochka17
Copy link

lanitochka17 commented Jul 5, 2024

If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!


Version Number: 9.0.4.0
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Issue reported by: Applause - Internal Team

Issue found when executing PR #44159

Action Performed:

  1. Create a connection with NetSuite on OD
  2. Open the account on ND
  3. Go to the workspace accounting settings
  4. Go to export configuration
  5. Go to Export reimbursable expenses as config
  6. Select Journal Entries for the Export as setting
  7. Open Reimbursable journal posting account
  8. Select any option if none is selected
  9. Do step 7 again and tap on any option without scrolling the list

Expected Result:

App selects the tapped option

Actual Result:

User is unable to select on the tapped option without scrolling first, user has to scroll first to be able to select from the options

Workaround:

Unknown

Platforms:

Which of our officially supported platforms is this issue occurring on?

  • Android: Native
  • Android: mWeb Chrome
  • iOS: Native
  • iOS: mWeb Safari
  • MacOS: Chrome / Safari
  • MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

Bug6534099_1720188001160.Screen_Recording_20240704_162916_New_Expensify.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~01315e9cb0864499cf
  • Upwork Job ID: 1810247323723969047
  • Last Price Increase: 2024-07-08
  • Automatic offers:
    • shubham1206agra | Contributor | 103033403
Issue OwnerCurrent Issue Owner: @akinwale
@lanitochka17 lanitochka17 added DeployBlockerCash This issue or pull request should block deployment Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. DeployBlocker Indicates it should block deploying the API labels Jul 5, 2024
@github-actions github-actions bot added Engineering Hourly KSv2 and removed Daily KSv2 labels Jul 5, 2024
Copy link
Contributor

github-actions bot commented Jul 5, 2024

👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open `StagingDeployCash` deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

  1. Identify the pull request that introduced this issue and revert it.
  2. Find someone who can quickly fix the issue.
  3. Fix the issue yourself.

Copy link

melvin-bot bot commented Jul 5, 2024

Triggered auto assignment to @arosiclair (DeployBlockerCash), see https://stackoverflowteams.com/c/expensify/questions/9980/ for more details.

Copy link

melvin-bot bot commented Jul 5, 2024

Triggered auto assignment to @kadiealexander (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.

@melvin-bot melvin-bot bot added Daily KSv2 and removed Hourly KSv2 labels Jul 5, 2024
@tgolen tgolen removed the DeployBlocker Indicates it should block deploying the API label Jul 5, 2024
@mountiny
Copy link
Contributor

mountiny commented Jul 5, 2024

I think Netsuite is still behind beta @yuwenmemon and this seems minor so we can demote and deploy

@mountiny
Copy link
Contributor

mountiny commented Jul 5, 2024

cc @tgolen

@yuwenmemon yuwenmemon removed the DeployBlockerCash This issue or pull request should block deployment label Jul 5, 2024
@kadiealexander
Copy link
Contributor

@arosiclair do you think this is something our contributors could work on?

@mountiny mountiny added the External Added to denote the issue can be worked on by a contributor label Jul 8, 2024
Copy link

melvin-bot bot commented Jul 8, 2024

Job added to Upwork: https://www.upwork.com/jobs/~01315e9cb0864499cf

@melvin-bot melvin-bot bot changed the title Android - NetSuite - Option is not being selected without scrolling first on a long list [$250] Android - NetSuite - Option is not being selected without scrolling first on a long list Jul 8, 2024
@melvin-bot melvin-bot bot added the Help Wanted Apply this label when an issue is open to proposals by contributors label Jul 8, 2024
Copy link

melvin-bot bot commented Jul 8, 2024

Triggered auto assignment to Contributor-plus team member for initial proposal review - @akinwale (External)

@mountiny
Copy link
Contributor

mountiny commented Jul 8, 2024

Yeah this can be external, but also tagging the involved engineers @yuwenmemon @mananjadhav

@ShridharGoel
Copy link
Contributor

@mountiny Are you able to replicate this?

@mountiny
Copy link
Contributor

mountiny commented Jul 8, 2024

I havent tried, but I can ask QA if they are still able to repro

@melvin-bot melvin-bot bot removed the Help Wanted Apply this label when an issue is open to proposals by contributors label Jul 8, 2024
Copy link

melvin-bot bot commented Jul 8, 2024

📣 @shubham1206agra 🎉 An offer has been automatically sent to your Upwork account for the Contributor role 🎉 Thanks for contributing to the Expensify app!

Offer link
Upwork job
Please accept the offer and leave a comment on the Github issue letting us know when we can expect a PR to be ready for review 🧑‍💻
Keep in mind: Code of Conduct | Contributing 📖

@yuwenmemon
Copy link
Contributor

@shubham1206agra can you please have a look at this one?

@melvin-bot melvin-bot bot added the Overdue label Jul 10, 2024
Copy link

melvin-bot bot commented Jul 11, 2024

@yuwenmemon, @shubham1206agra, @kadiealexander Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

Copy link

melvin-bot bot commented Jul 15, 2024

@yuwenmemon, @shubham1206agra, @kadiealexander Still overdue 6 days?! Let's take care of this!

@kadiealexander
Copy link
Contributor

@shubham1206agra bump!

@shubham1206agra
Copy link
Contributor

@yuwenmemon I am not sure what caused this.

@melvin-bot melvin-bot bot added the Overdue label Jul 18, 2024
Copy link

melvin-bot bot commented Jul 19, 2024

@yuwenmemon @shubham1206agra @kadiealexander this issue was created 2 weeks ago. Are we close to approving a proposal? If not, what's blocking us from getting this issue assigned? Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks!

Copy link

melvin-bot bot commented Jul 19, 2024

@yuwenmemon, @shubham1206agra, @kadiealexander Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

Copy link

melvin-bot bot commented Jul 23, 2024

@yuwenmemon, @shubham1206agra, @kadiealexander 6 days overdue. This is scarier than being forced to listen to Vogon poetry!

@yuwenmemon
Copy link
Contributor

No update yet for this, we are looking for outside help/input: https://expensify.slack.com/archives/C06ML6X0W9L/p1721715155581529?thread_ts=1721712356.211049&cid=C06ML6X0W9L

@melvin-bot melvin-bot bot removed the Overdue label Jul 23, 2024
@BrtqKr
Copy link
Contributor

BrtqKr commented Jul 26, 2024

Hey, I'm from SWM, I'd like to take care of this

@melvin-bot melvin-bot bot added the Overdue label Jul 26, 2024
@BrtqKr
Copy link
Contributor

BrtqKr commented Jul 26, 2024

One thing - how can I test NetSuite, do you have any accounts set up for debugging?

@yuwenmemon
Copy link
Contributor

@BrtqKr I can DM you some testing creds

@melvin-bot melvin-bot bot removed the Overdue label Jul 28, 2024
@yuwenmemon
Copy link
Contributor

@BrtqKr I can DM you some testing creds

@BrtqKr
Copy link
Contributor

BrtqKr commented Jul 29, 2024

@lanitochka17 @yuwenmemon From what I can see this seems to be working as expected, but I don't see any particular changes that could possibly be the reason why this got fixed in the meantime.

Screen.Recording.2024-07-29.at.15.35.35.mov

@melvin-bot melvin-bot bot added the Overdue label Jul 31, 2024
Copy link

melvin-bot bot commented Aug 1, 2024

@yuwenmemon, @shubham1206agra, @kadiealexander Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

@JmillsExpensify
Copy link

Going to close, so if anyone can reproduce please re-open the issue. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 Engineering External Added to denote the issue can be worked on by a contributor
Projects
Status: Done
Development

No branches or pull requests