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

[Hold for payment 2024-09-06] [#wave-control] [Bring copilot to NewDot] Display account switcher #46920

Closed
dangrous opened this issue Aug 6, 2024 · 15 comments
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 Reviewing Has a PR in review

Comments

@dangrous
Copy link
Contributor

dangrous commented Aug 6, 2024

Part of the Bring copilot to NewDot project

Main issue: https://github.com/Expensify/Expensify/issues/382866
Doc section: https://docs.google.com/document/d/18G6J5bl61VnVa5CoKpRXhM0MhkXDwpm87GVitZrSCVg/edit#heading=h.4gw8y2nhd2we
Project: https://github.com/orgs/Expensify/projects/130

Feature Description

Manual Test Steps

Automated Tests

@dangrous dangrous changed the title Display account switcher [#wave-control] [Bring copilot to NewDot] Display account switcher Aug 6, 2024
@dylanexpensify dylanexpensify added the Daily KSv2 label Aug 6, 2024
@dangrous dangrous self-assigned this Aug 7, 2024
@dangrous
Copy link
Contributor Author

dangrous commented Aug 7, 2024

assigning myself for review once it's ready

@melvin-bot melvin-bot bot added the Overdue label Aug 12, 2024
@rushatgabhane rushatgabhane mentioned this issue Aug 12, 2024
50 tasks
@melvin-bot melvin-bot bot added Reviewing Has a PR in review Weekly KSv2 and removed Daily KSv2 Overdue labels Aug 13, 2024
@melvin-bot melvin-bot bot added Weekly KSv2 and removed Weekly KSv2 labels Aug 13, 2024
@parasharrajat
Copy link
Member

Need access to the design doc @dangrous

@rushatgabhane
Copy link
Member

rushatgabhane commented Aug 28, 2024

Popover bug to fix LATER: #47338 (comment)

IOS: BUG: While offline, Alert does not show up on changing accounts.

@melvin-bot melvin-bot bot added Weekly KSv2 and removed Weekly KSv2 labels Aug 28, 2024
@dangrous
Copy link
Contributor Author

dangrous commented Aug 29, 2024

Okay just listing out the additional bugs found so far here. I think we expected there was going to be weirdness, so no worries, but definitely should look into it.

BUGS BEHIND BETA

BUGS CURRENTLY VISIBLE

The second section here is the priority, I think, but they're pretty much all visual quick fixes, nothing too crazy. @rushatgabhane do you think you could fix 'em all in one go?

The ones that are bugs with the copilot feature itself, I think we can probably just include those in the future copilot PRs, rather than doing them separately. Let me know if you disagree.

@rushatgabhane
Copy link
Member

@dangrous sounds good

@dangrous
Copy link
Contributor Author

dangrous commented Sep 4, 2024

@parasharrajat assigning you here since you reviewed, will grab a BZ for payment when ready!

@melvin-bot melvin-bot bot added Weekly KSv2 and removed Weekly KSv2 labels Sep 8, 2024
@parasharrajat
Copy link
Member

Can we please process the C+ payment here? @dangrous Please assign someone.

@dangrous
Copy link
Contributor Author

oops yep will do!

@dangrous dangrous added the Bug Something is broken. Auto assigns a BugZero manager. label Sep 23, 2024
Copy link

melvin-bot bot commented Sep 23, 2024

Triggered auto assignment to @sonialiap (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 Weekly KSv2 labels Sep 23, 2024
@dangrous
Copy link
Contributor Author

hey @sonialiap! @parasharrajat reviewed this PR as the C+, so will need payment; @rushatgabhane is getting handled at the project level so I believe no payment needed for them right now. We're through the 7 day period also!

@sonialiap
Copy link
Contributor

Thanks Dan :D

@sonialiap
Copy link
Contributor

went to production Aug 30, payment should have been Sep 6 😳

@sonialiap sonialiap changed the title [#wave-control] [Bring copilot to NewDot] Display account switcher [Hold for payment 2024-09-06] [#wave-control] [Bring copilot to NewDot] Display account switcher Sep 23, 2024
@sonialiap
Copy link
Contributor

Payment summary

@parasharrajat
Copy link
Member

Payment requested as per #46920 (comment)

@JmillsExpensify
Copy link

$250 approved for @parasharrajat

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 Reviewing Has a PR in review
Projects
Status: Done
Development

No branches or pull requests

6 participants