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 2022-12-20] Dev: TypeError on dev when workspace settings forms are opened #13344

Closed
kavimuru opened this issue Dec 5, 2022 · 16 comments
Assignees
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 Engineering Internal Requires API changes or must be handled by Expensify staff Reviewing Has a PR in review

Comments

@kavimuru
Copy link

kavimuru commented Dec 5, 2022

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:

  1. Go to Workspace settings.
  2. Click on “General settings.”

Alternately,

  1. Go to Workspace settings.
  2. Click on “Connect bank account.”
  3. Click on “Connect manually.”

Expected Result:

Form is opened normally

Actual Result:

Console error on desktop, full page blocking error on iOS (dev) and Android (dev)

Workaround:

unknown

Platform:

Where is this issue occurring?

  • Web
  • iOS
  • Android
  • Desktop App
  • Mobile Web

Version Number:
Reproducible in staging?: Need reproduction
Reproducible in production?: Need reproduction
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:

Expensify/Expensify Issue URL:
Issue reported by: @dangrous
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1670273493519649

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~01302e69a126221869
  • Upwork Job ID: 1605268377985359872
  • Last Price Increase: 2022-12-20
@kavimuru kavimuru added Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. Needs Reproduction Reproducible steps needed labels Dec 5, 2022
@luacmartins luacmartins self-assigned this Dec 5, 2022
@luacmartins
Copy link
Contributor

I can reproduce this and I'm working on a fix

@melvin-bot
Copy link

melvin-bot bot commented Dec 13, 2022

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

@melvin-bot melvin-bot bot added Weekly KSv2 Awaiting Payment Auto-added when associated PR is deployed to production and removed Daily KSv2 labels Dec 13, 2022
@melvin-bot melvin-bot bot changed the title Dev: TypeError on dev when workspace settings forms are opened [HOLD for payment 2022-12-20] Dev: TypeError on dev when workspace settings forms are opened Dec 13, 2022
@melvin-bot
Copy link

melvin-bot bot commented Dec 13, 2022

The solution for this issue has been 🚀 deployed to production 🚀 in version 1.2.38-6 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue:

If no regressions arise, payment will be issued on 2022-12-20. 🎊

After the hold period, please check if any of the following need payment for this issue, and if so check them off after paying:

  • External issue reporter
  • Contributor that fixed the issue
  • Contributor+ that helped on the issue and/or PR

As a reminder, here are the bonuses/penalties that should be applied for any External issue:

  • Merged PR within 3 business days of assignment - 50% bonus
  • Merged PR more than 9 business days after assignment - 50% penalty

@melvin-bot
Copy link

melvin-bot bot commented Dec 13, 2022

BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:

@luacmartins
Copy link
Contributor

We are good to close this one since there's no payment needed!

@mananjadhav
Copy link
Collaborator

@luacmartins There will be payment for the C+ review on PR #13348. Can you help with the same?

@luacmartins luacmartins reopened this Dec 20, 2022
@luacmartins luacmartins added Bug Something is broken. Auto assigns a BugZero manager. and removed Bug Something is broken. Auto assigns a BugZero manager. labels Dec 20, 2022
@melvin-bot
Copy link

melvin-bot bot commented Dec 20, 2022

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

@melvin-bot melvin-bot bot added Daily KSv2 and removed Weekly KSv2 labels Dec 20, 2022
@luacmartins
Copy link
Contributor

@mananjadhav thanks for bringing that up to my attention. Sorry that I missed that.

@maddylewis the only thing left to do here is pay @mananjadhav for this PR review. Could you help us with that?

@maddylewis
Copy link
Contributor

yes! let me do that now.

@luacmartins luacmartins added the Internal Requires API changes or must be handled by Expensify staff label Dec 20, 2022
@melvin-bot
Copy link

melvin-bot bot commented Dec 20, 2022

Job added to Upwork: https://www.upwork.com/jobs/~01302e69a126221869

@melvin-bot
Copy link

melvin-bot bot commented Dec 20, 2022

Triggered auto assignment to Contributor Plus for review of internal employee PR - @mananjadhav (Internal)

@maddylewis
Copy link
Contributor

okay, @mananjadhav - can you apply to the job and then i will process payment :)

@melvin-bot melvin-bot bot added Daily KSv2 and removed Daily KSv2 labels Dec 20, 2022
@mananjadhav
Copy link
Collaborator

Applied @maddylewis

@maddylewis
Copy link
Contributor

payment sent!

@maddylewis
Copy link
Contributor

ill go ahead and close this out. if there were any issues receiving payment, feel free to reopen and let me know - thanks, @mananjadhav !

@mananjadhav
Copy link
Collaborator

Received the payment. Thanks @maddylewis for the help here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 Engineering Internal Requires API changes or must be handled by Expensify staff Reviewing Has a PR in review
Projects
None yet
Development

No branches or pull requests

4 participants