Skip to content

[HOLD for payment 2023-10-23] [HOLD for payment 2023-10-23] [HOLD for payment 2023-10-20] [HOLD for payment 2023-10-20] [HOLD for payment 2023-10-18] [Wave 6: Drafts] Add the Submit button to the Report preview component #28766

Closed

Description

@mountiny

Part of https://github.com/Expensify/Expensify/issues/308213

Coming from the doc here, we need to add the report submit button to the report preview in the App.

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~01d291e7bf437cd92b
  • Upwork Job ID: 1709582242081632256
  • Last Price Increase: 2023-10-04

Activity

  1. changed the title [-]Wave6 - Drafts - Add the Submit button to the Report preview component[/-] [+][Wave 6: Drafts] Add the Submit button to the Report preview component[/+] on Oct 4, 2023
  2. BeeMargarida commented on Oct 4, 2023

    @BeeMargarida
    Contributor

    Hey @mountiny, Ana from Callstack here 👋, I'm would like to work on this issue

  3. melvin-bot commented on Oct 4, 2023

    @melvin-bot

    Current assignee @puneetlath is eligible for the NewFeature assigner, not assigning anyone new.

  4. added
    InternalRequires API changes or must be handled by Expensify staff
    on Oct 4, 2023
  5. melvin-bot commented on Oct 4, 2023

    @melvin-bot
  6. melvin-bot commented on Oct 4, 2023

    @melvin-bot

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

  7. 22 remaining items

  8. changed the title [-][HOLD for payment 2023-10-20] [HOLD for payment 2023-10-18] [Wave 6: Drafts] Add the Submit button to the Report preview component[/-] [+][HOLD for payment 2023-10-20] [HOLD for payment 2023-10-20] [HOLD for payment 2023-10-18] [Wave 6: Drafts] Add the Submit button to the Report preview component[/+] on Oct 13, 2023
  9. melvin-bot commented on Oct 13, 2023

    @melvin-bot

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

    After the hold period is over and BZ checklist items are completed, please complete any of the applicable payments for this issue, and check them off once done.

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

    For reference, here are some details about the assignees on this issue:

    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
  10. mountiny commented on Oct 15, 2023

    @mountiny
    ContributorAuthor

    No payment required, this is completed

  11. changed the title [-][HOLD for payment 2023-10-20] [HOLD for payment 2023-10-20] [HOLD for payment 2023-10-18] [Wave 6: Drafts] Add the Submit button to the Report preview component[/-] [+][HOLD for payment 2023-10-23] [HOLD for payment 2023-10-20] [HOLD for payment 2023-10-20] [HOLD for payment 2023-10-18] [Wave 6: Drafts] Add the Submit button to the Report preview component[/+] on Oct 16, 2023
  12. melvin-bot commented on Oct 16, 2023

    @melvin-bot

    The solution for this issue has been 🚀 deployed to production 🚀 in version 1.3.84-10 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 2023-10-23. 🎊

    After the hold period is over and BZ checklist items are completed, please complete any of the applicable payments for this issue, and check them off once done.

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

    For reference, here are some details about the assignees on this issue:

    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
  13. changed the title [-][HOLD for payment 2023-10-23] [HOLD for payment 2023-10-20] [HOLD for payment 2023-10-20] [HOLD for payment 2023-10-18] [Wave 6: Drafts] Add the Submit button to the Report preview component[/-] [+][HOLD for payment 2023-10-23] [HOLD for payment 2023-10-23] [HOLD for payment 2023-10-20] [HOLD for payment 2023-10-20] [HOLD for payment 2023-10-18] [Wave 6: Drafts] Add the Submit button to the Report preview component[/+] on Oct 16, 2023
  14. melvin-bot commented on Oct 16, 2023

    @melvin-bot

    The solution for this issue has been 🚀 deployed to production 🚀 in version 1.3.84-10 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 2023-10-23. 🎊

    After the hold period is over and BZ checklist items are completed, please complete any of the applicable payments for this issue, and check them off once done.

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

    For reference, here are some details about the assignees on this issue:

    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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

Awaiting PaymentAuto-added when associated PR is deployed to productionInternalRequires API changes or must be handled by Expensify staffNewFeatureSomething to build that is a new item.WeeklyKSv2

Type

No type

Projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions