-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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-06-28] [Payment card / Subscription] Integrate “Subscription settings” section with backend data #38627
Comments
Hi, I'm Fábio from Callstack - expert agency - and I would like to work on this issue. |
Hi, I'm Julian from Callstack - expert agency - and I would like to work on this issue. |
@fabioh8010 Can you help me update the body of this issue to reflect exactly which part of the doc it covers please? Thanks |
@MitchExpensify Is the section you linked in this GH issue, |
Cool so the body is linking to the correct section of the doc, just to be sure, right? @fabioh8010 |
I think it makes the most sense to implement all toggle actions as a part of this issue. Backend integration for that section would be a very tiny PR, like 2-3 lines of new code, so I think it would just bring more work for the reviewers to have 2 separate tasks. Wdyt? I'm already working on it and will be posting a PR soon CC: @amyevans |
Yep that sounds great! Thanks @JKobrynski! |
Reassigning to @blimpich as I'm headed out on extended leave |
|
The solution for this issue has been 🚀 deployed to production 🚀 in version 9.0.0-9 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 2024-06-28. 🎊 For reference, here are some details about the assignees on this issue:
|
Skipping the payment summary for this issue since all the assignees are employees or vendors. If this is incorrect, please manually add the payment summary SO. |
@blimpich, @JKobrynski Whoops! This issue is 2 days overdue. Let's get this updated quick! |
No BZ on this either. Payment summary as follows:
New feature, regression tests are going to be added centrally for the project as we wrap up soon. I'll send you an offer on Upwork now. |
@trjExpensify As this Slack thread,
this issue was created on March 20 when the initial price still was 500$. Should we update the offer to increase the compensation? |
Ah yeah, fair enough. Missed that. Accept and I'll adjust. |
@trjExpensify Thanks 🙏 I accepted it |
Paid, closing! |
Covers this section of the doc: https://docs.google.com/document/d/1jXV_vIsOGZuL0dMiuXDWRETAFRX64ODVAhWPtiuFXT8/edit?pli=1#bookmark=id.fwc9tpz3pb79
cc @fabioh8010
The text was updated successfully, but these errors were encountered: