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

[Payment due] Android - Skeleton and arrow appears when login via concierge deep link #39698

Closed
1 of 6 tasks
kbecciv opened this issue Apr 5, 2024 · 42 comments
Closed
1 of 6 tasks
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 External Added to denote the issue can be worked on by a contributor Reviewing Has a PR in review

Comments

@kbecciv
Copy link

kbecciv commented Apr 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: 1.4.59-0
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/4472211&group_by=cases:section_id&group_order=asc&group_id=229066
Issue reported by: Applause - Internal Testing

Action Performed:

  1. Navigate to https://staging.new.expensify.com/concierge
    from external app
  2. Sign-up with a new account when app opens

Expected Result:

User should navigate to the concierge chat

Actual Result:

Before Concierge page open skeleton and arrow appears on the center of the screen when login via concierge deep link. Also, when keyboard overlaps Welcome to Expensify modal

Workaround:

n/a

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

Bug6435953_1712085158199.az_recorder_20240402_210041.mp4

View all open jobs on GitHub

Issue OwnerCurrent Issue Owner: @dylanexpensify
@kbecciv kbecciv added Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels Apr 5, 2024
Copy link

melvin-bot bot commented Apr 5, 2024

Triggered auto assignment to @dylanexpensify (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

@kbecciv
Copy link
Author

kbecciv commented Apr 5, 2024

@dylanexpensify I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors.

@melvin-bot melvin-bot bot added the Overdue label Apr 8, 2024
Copy link

melvin-bot bot commented Apr 8, 2024

@dylanexpensify Whoops! This issue is 2 days overdue. Let's get this updated quick!

@dylanexpensify
Copy link
Contributor

reviewing today!

@melvin-bot melvin-bot bot added Overdue and removed Overdue labels Apr 9, 2024
@dylanexpensify
Copy link
Contributor

Browserstack was being funny for me, trying again today!

@melvin-bot melvin-bot bot removed the Overdue label Apr 11, 2024
@dylanexpensify
Copy link
Contributor

Unable to repro - @kbecciv can you confirm you can't repro on latest?

@melvin-bot melvin-bot bot added the Overdue label Apr 15, 2024
@dylanexpensify
Copy link
Contributor

Bump @kbecciv 🙇‍♂️

@melvin-bot melvin-bot bot added Overdue and removed Overdue labels Apr 15, 2024
Copy link

melvin-bot bot commented Apr 18, 2024

@dylanexpensify Whoops! This issue is 2 days overdue. Let's get this updated quick!

Copy link

melvin-bot bot commented Apr 19, 2024

@dylanexpensify this issue was created 2 weeks ago. Are we close to a solution? Let's make sure we're treating this as a top priority. Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks!

Copy link

melvin-bot bot commented Apr 22, 2024

@dylanexpensify 6 days overdue. This is scarier than being forced to listen to Vogon poetry!

@kbecciv
Copy link
Author

kbecciv commented Apr 23, 2024

@dylanexpensify Checking

@kbecciv
Copy link
Author

kbecciv commented Apr 23, 2024

Issue is no longer reproduced

az_recorder_20240423_204955.mp4

Copy link

melvin-bot bot commented Apr 24, 2024

@dylanexpensify 8 days overdue is a lot. Should this be a Weekly issue? If so, feel free to change it!

@lanitochka17
Copy link

Issue is still reproducible

44063.iOS.MP4

@lanitochka17 lanitochka17 reopened this Jun 26, 2024
@melvin-bot melvin-bot bot added the Overdue label Jun 28, 2024
Copy link

melvin-bot bot commented Jul 1, 2024

@dylanexpensify Huh... This is 4 days overdue. Who can take care of this?

@dylanexpensify dylanexpensify added the External Added to denote the issue can be worked on by a contributor label Jul 3, 2024
@melvin-bot melvin-bot bot removed the Overdue label Jul 15, 2024
@nkdengineer
Copy link
Contributor

@yuwenmemon Friendly bump for assignment #39698 (comment)

Thanks 🙇

@melvin-bot melvin-bot bot removed the Help Wanted Apply this label when an issue is open to proposals by contributors label Jul 15, 2024
@melvin-bot melvin-bot bot added Reviewing Has a PR in review Weekly KSv2 and removed Daily KSv2 labels Jul 16, 2024
@mvtglobally
Copy link

Issue not reproducible during KI retests. (First week)

@dylanexpensify
Copy link
Contributor

@nkdengineer can we get an update please?

@nkdengineer
Copy link
Contributor

Screen.Recording.2024-08-05.at.21.36.54.mov

@dylanexpensify this bug still reproduce, and PR is under reviewing

@melvin-bot melvin-bot bot removed the Weekly KSv2 label Aug 8, 2024
Copy link

melvin-bot bot commented Aug 8, 2024

This issue has not been updated in over 15 days. @yuwenmemon, @sobitneupane, @dylanexpensify, @nkdengineer 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 Aug 8, 2024
@sobitneupane
Copy link
Contributor

The PR is being reviewed.

@sobitneupane
Copy link
Contributor

@dylanexpensify PR deployed to production two weeks back. #45460 (comment)

@nkdengineer
Copy link
Contributor

@dylanexpensify Gentle bump for payment, TIA

@dylanexpensify
Copy link
Contributor

Whoops - automation failed!

@dylanexpensify
Copy link
Contributor

Payment summary:

Contributor: @nkdengineer $250 via Upwork
Contributor+: @sobitneupane $250 via NewDot

Please apply/request!

@dylanexpensify dylanexpensify changed the title Android - Skeleton and arrow appears when login via concierge deep link [Payment due] Android - Skeleton and arrow appears when login via concierge deep link Oct 7, 2024
@dylanexpensify dylanexpensify added Daily KSv2 and removed Monthly KSv2 labels Oct 7, 2024
@dylanexpensify
Copy link
Contributor

@nkdengineer apply here! I sent invite too!

@dylanexpensify
Copy link
Contributor

pending!

@nkdengineer
Copy link
Contributor

@dylanexpensify I applied

@dylanexpensify
Copy link
Contributor

@nkdengineer offer sent!

@nkdengineer
Copy link
Contributor

@dylanexpensify I accepted it

@dylanexpensify
Copy link
Contributor

Done!

@garrettmknight
Copy link
Contributor

$250 approved for @sobitneupane

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 External Added to denote the issue can be worked on by a contributor Reviewing Has a PR in review
Projects
No open projects
Status: No status
Development

No branches or pull requests

8 participants