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 2023-12-21][$500] Update CONSTS to direct developers to use role instead of accessibilityRole #31500

Closed
dangrous opened this issue Nov 17, 2023 · 45 comments
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Engineering External Added to denote the issue can be worked on by a contributor Reviewing Has a PR in review Weekly KSv2

Comments

@dangrous
Copy link
Contributor

dangrous commented Nov 17, 2023

  • Restore CONST.ACCESSIBLITY_ROLE.TEXT to have the value text
  • Create CONST.ROLE with comments on appropriate use
  • Add a deprecation JSDoc on CONST.ACCESSIBLITY_ROLE with an instruction to use role instead of accessibilityRole
  • Update any role props that point to CONST.ACCESSIBLITY_ROLE to point to CONST.ROLE instead.

This will encourage appropriate code syntax moving forward and keep things clean and on the cutting edge!

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~012561c8136f1b8b5a
  • Upwork Job ID: 1726618160989405184
  • Last Price Increase: 2023-11-20
Issue OwnerCurrent Issue Owner: @JmillsExpensify
@dangrous dangrous changed the title Update CONSTS to direct users to use role instead of accessibilityRole Update CONSTS to direct developers to use role instead of accessibilityRole Nov 17, 2023
@dangrous dangrous self-assigned this Nov 17, 2023
@trevor-coleman
Copy link
Contributor

trevor-coleman commented Nov 17, 2023

Hey @dangrous -- this is my baby!

Copy link

melvin-bot bot commented Nov 17, 2023

📣 @trevor-coleman! 📣
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>

@trevor-coleman
Copy link
Contributor

As a note: I don't get paid via Upwork as I'm with Infinite Red.

@dangrous dangrous added Engineering Weekly KSv2 External Added to denote the issue can be worked on by a contributor labels Nov 20, 2023
@melvin-bot melvin-bot bot changed the title Update CONSTS to direct developers to use role instead of accessibilityRole [$500] Update CONSTS to direct developers to use role instead of accessibilityRole Nov 20, 2023
Copy link

melvin-bot bot commented Nov 20, 2023

Job added to Upwork: https://www.upwork.com/jobs/~012561c8136f1b8b5a

@melvin-bot melvin-bot bot added the Help Wanted Apply this label when an issue is open to proposals by contributors label Nov 20, 2023
Copy link

melvin-bot bot commented Nov 20, 2023

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

@melvin-bot melvin-bot bot added Daily KSv2 and removed Weekly KSv2 labels Nov 20, 2023
@melvin-bot melvin-bot bot removed the Help Wanted Apply this label when an issue is open to proposals by contributors label Nov 20, 2023
Copy link

melvin-bot bot commented Nov 20, 2023

📣 @allroundexperts Please request via NewDot manual requests for the Contributor role ($500)

Copy link

melvin-bot bot commented Nov 20, 2023

📣 @allroundexperts Please request via NewDot manual requests for the Reviewer role ($500)

Copy link

melvin-bot bot commented Nov 20, 2023

The BZ member will need to manually hire trevor-coleman for the Contributor role. Please store your Upwork details and apply to our Upwork job so this process is automatic in the future!

@dangrous
Copy link
Contributor Author

sorry for all the notifications - trying to get a C+ here, (thanks @allroundexperts!), @trevor-coleman will be handling the issue

trevor-coleman added a commit to infinitered/ExpensifyApp that referenced this issue Nov 20, 2023
Update CONSTS to direct developers to use role instead of accessibilityRole (Expensify#31500)
@trevor-coleman
Copy link
Contributor

I've got a draft PR going, but not sure how to handle the testing part.

Should I write up tests for every component touched her?

It would be quite a few components (~100), and as the changes are a like-for-like substitution of values from a known list of acceptable values I think the risk it pretty low.

Happy to do whatever is needed, just wanted to get some advice on what an appropriate testing section looks like for a find-replace PR like this.

@dangrous
Copy link
Contributor Author

That's a great question. I think if most of the changes are exactly the same, I think we can call out the testing steps for just a few of them. Since this also shouldn't cause any visual change, it shouldn't be too bad.

@allroundexperts since you've done a lot more testing, what are your thoughts on the best way forward?

@getusha
Copy link
Contributor

getusha commented Nov 21, 2023

@trevor-coleman @dangrous what about adding a new lint rule?

@abekkala
Copy link
Contributor

PAYOUT NOTES
$500
@allroundexperts paid in NEWDOT, not Upwork

@dangrous you can close this when needed as @allroundexperts will request his payment via NewDot

Copy link

melvin-bot bot commented Dec 21, 2023

Issue is ready for payment but no BZ is assigned. @peterdbarkerUK you are the lucky winner! Please verify the payment summary looks correct and complete the checklist. Thanks!

Copy link

melvin-bot bot commented Dec 21, 2023

### Payment Summary
[Upwork Job](https://www.upwork.com/jobs/~012561c8136f1b8b5a)
- <Role>: @trevor-coleman paid $(<amount>) via Upwork (<Replace with offer link>)

@peterdbarkerUK
Copy link
Contributor

peterdbarkerUK commented Dec 21, 2023

@trevor-coleman doesn't need to be paid via UpWork, as he's Infinite Red.

@allroundexperts could you send your request via NewDot? @JmillsExpensify is OoO so it might take a week or so to action: $500 for C+ review.

@peterdbarkerUK
Copy link
Contributor

Weird that this assigned me for the payment issue in addition to @abekkala, maybe due to holiday season OoO?

@peterdbarkerUK
Copy link
Contributor

@JmillsExpensify $500 to @allroundexperts when it's sent, thanks!

Copy link

melvin-bot bot commented Jan 3, 2024

@JmillsExpensify, @dangrous, @abekkala, @peterdbarkerUK, @trevor-coleman, @allroundexperts Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

@abekkala
Copy link
Contributor

abekkala commented Jan 3, 2024

@allroundexperts have you requested your payment in New Dot?

@abekkala
Copy link
Contributor

abekkala commented Jan 4, 2024

This one is ready to be closed out once:
@allroundexperts requests $500 payment in New Dot

@JmillsExpensify

Copy link

melvin-bot bot commented Jan 5, 2024

@JmillsExpensify, @dangrous, @abekkala, @peterdbarkerUK, @trevor-coleman, @allroundexperts Eep! 4 days overdue now. Issues have feelings too...

@allroundexperts
Copy link
Contributor

@abekkala I've noted it down. Feel free to close. Thanks!

Copy link

melvin-bot bot commented Jan 9, 2024

@JmillsExpensify, @dangrous, @abekkala, @peterdbarkerUK, @trevor-coleman, @allroundexperts Now this issue is 8 days overdue. Are you sure this should be a Daily? Feel free to change it!

@peterdbarkerUK
Copy link
Contributor

@allroundexperts sorry Sibtain, I'm a bit confused: did you already request and receive this payment?

Copy link

melvin-bot bot commented Jan 11, 2024

@JmillsExpensify, @dangrous, @abekkala, @peterdbarkerUK, @trevor-coleman, @allroundexperts 10 days overdue. Is anyone even seeing these? Hello?

@allroundexperts
Copy link
Contributor

@peterdbarkerUK I have requested payment but haven't received it yet. I think we can close this issue because receiving the payment does take some time.

@peterdbarkerUK
Copy link
Contributor

CC @JmillsExpensify this one should be ready to go, $500 to @allroundexperts

Let's close once payment is confirmed, no need to risk things slipping through the cracks :)

@melvin-bot melvin-bot bot added Weekly KSv2 and removed Daily KSv2 labels Jan 16, 2024
Copy link

melvin-bot bot commented Jan 16, 2024

This issue has not been updated in over 14 days. @JmillsExpensify, @dangrous, @abekkala, @peterdbarkerUK, @trevor-coleman, @allroundexperts eroding to Weekly issue.

@dangrous
Copy link
Contributor Author

How are we looking on this one? I think still waiting on payment for @allroundexperts from @JmillsExpensify, is that right?

@abekkala
Copy link
Contributor

How are we looking on this one? I think still waiting on payment for @allroundexperts from @JmillsExpensify, is that right?

yes, that is correct

@JmillsExpensify
Copy link

I don't think I've received a payment request, though as a heads up, we don't keep issues open that are waiting payment in Expensify. Feel free to close it.

@JmillsExpensify
Copy link

Ah yeah, I just checked. @allroundexperts needs to submit the report to me, as that's the only way it shows up in my LHN. But again, no need to keep this issue open for Expensify payments, so I'd go ahead and close it.

@abekkala
Copy link
Contributor

sounds good @JmillsExpensify thanks!

@JmillsExpensify
Copy link

$500 payment to @allroundexperts based on this comment.

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. Engineering External Added to denote the issue can be worked on by a contributor Reviewing Has a PR in review Weekly KSv2
Projects
None yet
Development

No branches or pull requests

8 participants