Skip to content

[Collect Approvers] Add Approve and Pay buttons by using report NextSteps #29145

Closed

Description

@marcochavezf

On hold for https://github.com/Expensify/Expensify/issues/323548
On hold now for https://github.com/Expensify/Web-Expensify/pull/39328

Coming from this internal discussion, we're going to use the report NextSteps data from OldDot to render the Approve or Pay button for collect policies according to the user.

Additionally, we want to render the "approved" message in the corresponding places listed in the HL, like in the report header, report preview and the expense report.

Activity

  1. self-assigned this
    on Oct 10, 2023
  2. marcochavezf commented on Oct 11, 2023

    @marcochavezf
    ContributorAuthor

    Fixed more edge cases that I've found but in the frontend. Tomorrow I will run another set of local tests to prepare the PR for review.

  3. changed the title [-][HOLD][Collect Approvers] Add Approve and Pay buttons by using report NextSteps[/-] [+][Collect Approvers] Add Approve and Pay buttons by using report NextSteps[/+] on Oct 11, 2023
  4. marcochavezf commented on Oct 12, 2023

    @marcochavezf
    ContributorAuthor

    PR in review

  5. marcochavezf commented on Oct 17, 2023

    @marcochavezf
    ContributorAuthor
  6. marcochavezf commented on Oct 24, 2023

    @marcochavezf
    ContributorAuthor

    I'm working on the backend PRs, more details here.

  7. changed the title [-][Collect Approvers] Add Approve and Pay buttons by using report NextSteps[/-] [+][HOLD Web-E #39328][Collect Approvers] Add Approve and Pay buttons by using report NextSteps[/+] on Oct 24, 2023
  8. changed the title [-][HOLD Web-E #39328][Collect Approvers] Add Approve and Pay buttons by using report NextSteps[/-] [+][Collect Approvers] Add Approve and Pay buttons by using report NextSteps[/+] on Nov 13, 2023
  9. 9 remaining items

  10. shubham1206agra commented on Dec 13, 2023

    @shubham1206agra
    Contributor

    Please assign me here as C+

  11. melvin-bot commented on Dec 14, 2023

    @melvin-bot

    @marcochavezf Eep! 4 days overdue now. Issues have feelings too...

  12. melvin-bot commented on Dec 18, 2023

    @melvin-bot

    @marcochavezf Now this issue is 8 days overdue. Are you sure this should be a Daily? Feel free to change it!

  13. melvin-bot commented on Dec 20, 2023

    @melvin-bot
  14. marcochavezf commented on Dec 20, 2023

    @marcochavezf
    ContributorAuthor

    Hi @dylanexpensify, could you handle the payment for @shubham1206agra for #28976? 🙇🏽

  15. dylanexpensify commented on Dec 20, 2023

    @dylanexpensify
    Contributor

    Yes, will do!

  16. dylanexpensify commented on Dec 20, 2023

    @dylanexpensify
    Contributor

    @shubham1206agra please apply here!

  17. shubham1206agra commented on Dec 20, 2023

    @shubham1206agra
    Contributor

    Applied

  18. marcochavezf commented on Dec 21, 2023

    @marcochavezf
    ContributorAuthor

    Ok, I think we can close it out. Thanks! Also, the PR hit production last week

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions