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-11-29] [N7 Account Settings] Create Pronouns page #11576

Closed
Beamanator opened this issue Oct 4, 2022 · 25 comments
Closed
Assignees
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Engineering Improvement Item broken or needs improvement. Internal Requires API changes or must be handled by Expensify staff Reviewing Has a PR in review Weekly KSv2

Comments

@Beamanator
Copy link
Contributor

Beamanator commented Oct 4, 2022

Tracking issue: https://github.com/Expensify/Expensify/issues/182122

Detailed section of doc here: https://docs.google.com/document/d/17akqrbC-w8FqEoDLvcLmrX6iw4uaVrT_xlwAl-IhEzE/edit#heading=h.5n8w70bpe690

Note: This can only be implemented after UpdatePronouns command has been built - see https://github.com/Expensify/Expensify/issues/238724

@Beamanator Beamanator added Engineering Weekly KSv2 Improvement Item broken or needs improvement. labels Oct 4, 2022
@Beamanator Beamanator self-assigned this Oct 4, 2022
@Beamanator Beamanator added the Internal Requires API changes or must be handled by Expensify staff label Oct 5, 2022
@melvin-bot melvin-bot bot added the Overdue label Oct 12, 2022
@Beamanator
Copy link
Contributor Author

On hold for WhatsApp Quality

@melvin-bot melvin-bot bot removed the Overdue label Oct 14, 2022
@puneetlath puneetlath added the Bug Something is broken. Auto assigns a BugZero manager. label Oct 19, 2022
@melvin-bot melvin-bot bot added the Overdue label Oct 24, 2022
@Beamanator
Copy link
Contributor Author

Still on hold for now, might pick this up next week as discussions from here (#11795) are indicating this could be useful to implement sooner rather than later

@melvin-bot melvin-bot bot removed the Overdue label Oct 26, 2022
@Beamanator Beamanator removed the Bug Something is broken. Auto assigns a BugZero manager. label Oct 26, 2022
@Beamanator Beamanator changed the title [HOLD] [N7 Account Settings] Create Pronouns page [N7 Account Settings] Create Pronouns page Oct 27, 2022
@Beamanator Beamanator changed the title [N7 Account Settings] Create Pronouns page [HOLD] [N7 Account Settings] Create Pronouns page Oct 27, 2022
@Beamanator
Copy link
Contributor Author

Still on hold, need to discuss if the dropdown should be push-to-page to be consistent with other dropdowns (like new timezone selector)

@Beamanator Beamanator changed the title [HOLD] [N7 Account Settings] Create Pronouns page [HOLD Web-E#35315] [N7 Account Settings] Create Pronouns page Oct 31, 2022
@Beamanator
Copy link
Contributor Author

Beamanator commented Oct 31, 2022

Agreed here that we're going with the current design of the Pronouns page in the doc:

we will continue to support dropdowns in the case of lists that exists within a grouped set of fields.

@Beamanator Beamanator mentioned this issue Oct 31, 2022
92 tasks
@Beamanator Beamanator added the Reviewing Has a PR in review label Nov 1, 2022
@Beamanator
Copy link
Contributor Author

FYI we now may go with the current Pronouns design & open a new, separate project for redesigning the page & deciding what kind of "custom pronoun selector / creator" we want to implement (discussion)

@Beamanator Beamanator changed the title [HOLD Web-E#35315] [N7 Account Settings] Create Pronouns page [N7 Account Settings] Create Pronouns page Nov 10, 2022
@Beamanator
Copy link
Contributor Author

@melvin-bot melvin-bot bot closed this as completed Nov 15, 2022
@thesahindia
Copy link
Member

@Beamanator, C+ compensation is pending on this one.

@Beamanator Beamanator added the Bug Something is broken. Auto assigns a BugZero manager. label Nov 21, 2022
@melvin-bot
Copy link

melvin-bot bot commented Nov 21, 2022

Triggered auto assignment to @mateocole (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 Nov 21, 2022
@Beamanator
Copy link
Contributor Author

Hey @mateocole can you pay out @thesahindia for his work reviewing the related internal PR? 👍

@Beamanator Beamanator reopened this Nov 21, 2022
@Beamanator
Copy link
Contributor Author

(reopening till payment has been completed)

@flodnv flodnv removed the Bug Something is broken. Auto assigns a BugZero manager. label Nov 21, 2022
@mateocole
Copy link

hey @Beamanator from reading the S/O is there an upwork associated with the PR review? it looks like i need to make sure i pay through that

@thesahindia
Copy link
Member

hey @Beamanator from reading the S/O is there an upwork associated with the PR review

We will need to create one (no job right now)

If I am not wrong we just need to add external label and melvin will post the upwork job link on this issue.

@Beamanator
Copy link
Contributor Author

@mateocole yes as you mentioned, we'll need an Upwork job for this. And interesting point @thesahindia , I know that automation is new, I don't necessarily like the idea of needing to add External label since this will also assign a C+ (which isn't needed for this issue) right? I'll ask internally, I think since all of this is new we'll eventually come up with better automation for this exact case :D

@thesahindia
Copy link
Member

Yeah, makes sense!

@melvin-bot melvin-bot bot added Weekly KSv2 Awaiting Payment Auto-added when associated PR is deployed to production and removed Daily KSv2 labels Nov 22, 2022
@melvin-bot melvin-bot bot changed the title [N7 Account Settings] Create Pronouns page [HOLD for payment 2022-11-29] [N7 Account Settings] Create Pronouns page Nov 22, 2022
@melvin-bot
Copy link

melvin-bot bot commented Nov 22, 2022

The solution for this issue has been 🚀 deployed to production 🚀 in version 1.2.29-7 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-11-29. 🎊

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 melvin-bot bot added Daily KSv2 and removed Weekly KSv2 labels Nov 29, 2022
@thesahindia
Copy link
Member

Bump @mateocole for upwork

@mateocole
Copy link

@thesahindia
Copy link
Member

@mateocole, I have applied and increased the job price since compensation for internal PR review is $1000 (slack post, please see the 4th point)

@Beamanator
Copy link
Contributor Author

Hmm it looks like @thesahindia you reviewed on Nov 15 (here) but the update to increase C+ bounty to $1000 was made on Nov 18... I'll leave it up to @mateocole / @michaelhaxhiu to decide, I'm just pointing out date diff :D

@thesahindia
Copy link
Member

Oh correct, I didn't check the date 😅

@mateocole
Copy link

Hey @thesahindia, would you be able to withdraw the proposal at the current rate, and go back to the Nov 15 rate?

@thesahindia
Copy link
Member

@mateocole, I have changed the terms.

@mateocole
Copy link

Okay, offer sent!

@thesahindia
Copy link
Member

Accepted, thanks!

@mateocole
Copy link

Okay, payment sent!

@melvin-bot melvin-bot bot added Weekly KSv2 and removed Daily KSv2 labels Apr 3, 2024
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 Engineering Improvement Item broken or needs improvement. Internal Requires API changes or must be handled by Expensify staff Reviewing Has a PR in review Weekly KSv2
Projects
None yet
Development

No branches or pull requests

5 participants