Skip to content

[HOLD] [TS migration] Migrate 'themes' style to TypeScript #24731

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/styles/themes/useThemePreference.js 4
src/styles/themes/useTheme.js 2
src/styles/themes/default.js 2
src/styles/themes/ThemeProvider.js 7
src/styles/themes/ThemeContext.js 2

Activity

  1. fabioh8010 commented on Aug 28, 2023

    @fabioh8010
    Contributor

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

  2. fabioh8010 commented on Aug 28, 2023

    @fabioh8010
    Contributor

    @hayata-suenaga Let's put this on HOLD because of this PR -> #24178

  3. changed the title [-][TS migration] Migrate 'themes' style to TypeScript[/-] [+][HOLD] [TS migration] Migrate 'themes' style to TypeScript[/+] on Aug 28, 2023
  4. melvin-bot commented on Sep 11, 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!

  5. 2 remaining items

  6. melvin-bot commented on Oct 5, 2023

    @melvin-bot
    Author

    This issue has not been updated in over 15 days. @fabioh8010 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!

  7. hayata-suenaga commented on Oct 6, 2023

    @hayata-suenaga
    Contributor

    back to weekly. We're just waiting for theme switching project to finish

  8. melvin-bot commented on Oct 30, 2023

    @melvin-bot
    Author

    This issue has not been updated in over 15 days. @fabioh8010 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!

  9. hayata-suenaga commented on Oct 30, 2023

    @hayata-suenaga
    Contributor

    back to weekly. We're just waiting for theme switching project to finish

  10. fabioh8010 commented on Nov 15, 2023

    @fabioh8010
    Contributor

    @hayata-suenaga #24178 was merged, we can close this one now!

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

Metadata

Metadata

Assignees

Type

No type

Projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions