Skip to content

[HOLD for payment 2024-03-20] [TS migration] Migrate 'EReceipt.stories.js' story to TypeScript #32039

Closed

Description

@melvin-bot

TypeScript migration

Make sure you read through our TypeScript's style guide, cheatsheet and PropTypes conversion table before you start working on this migration issue.

Files

Path Dependencies
src/stories/EReceipt.stories.js 4

Activity

  1. melvin-bot commented on Dec 21, 2023

    @melvin-bot
    Author

    This issue has not been updated in over 15 days. eroding to Monthly issue.

    P.S. Is everyone reading this sure this is really a near-term priority? Be brave: if you disagree, go ahead and close it out. If someone disagrees, they'll reopen it, and if they don't: one less thing to do!

  2. JKobrynski commented on Mar 6, 2024

    @JKobrynski
    Contributor

    Hi, I'm Julian from Callstack - expert contributor group - and I would like to take a look at this issue.

  3. JKobrynski commented on Mar 11, 2024

    @JKobrynski
    Contributor

    The PR is ready for review!

  4. self-assigned this
    on Mar 12, 2024
  5. 7 remaining items

  6. changed the title [-][TS migration] Migrate 'EReceipt.stories.js' story to TypeScript[/-] [+][HOLD for payment 2024-03-20] [TS migration] Migrate 'EReceipt.stories.js' story to TypeScript[/+] on Mar 13, 2024
  7. melvin-bot commented on Mar 13, 2024

    @melvin-bot
    Author

    Reviewing label has been removed, please complete the "BugZero Checklist".

  8. melvin-bot commented on Mar 13, 2024

    @melvin-bot
    Author

    The solution for this issue has been 🚀 deployed to production 🚀 in version 1.4.51-3 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-03-20. 🎊

  9. melvin-bot commented on Mar 13, 2024

    @melvin-bot
    Author

    BugZero Checklist: The PR adding this new feature has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:

    • [@Julesssss] Please propose regression test steps to ensure the new feature will work correctly on production in further releases.
    • [] Link the GH issue for creating/updating the regression test once above steps have been agreed upon.
  10. Julesssss commented on Mar 14, 2024

    @Julesssss
    Contributor

    No test needed, this was a TS refactor

  11. melvin-bot commented on Mar 20, 2024

    @melvin-bot
    Author

    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.

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

Metadata

Metadata

Assignees

Labels

Type

No type

Projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions