Skip to content
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

fix: fix type issues on react-email templates on express server #7279

Conversation

karrui
Copy link
Contributor

@karrui karrui commented Apr 23, 2024

TL;DR

Fix type errors on frontend since root types are of a higher version.

@karrui karrui requested review from justynoh and KenLSM April 23, 2024 07:28
@karrui karrui marked this pull request as ready for review April 23, 2024 07:28
@karrui karrui force-pushed the 04-23-feat_upgrade_prettier_to_v3 branch from 323bcc0 to b5ef4d3 Compare April 23, 2024 07:49
@karrui karrui force-pushed the 04-23-fix_remove_erroneous_react_type_pkg_install_on_root branch from 3f918dd to 42e4267 Compare April 23, 2024 07:50
@karrui karrui force-pushed the 04-23-feat_upgrade_prettier_to_v3 branch from b5ef4d3 to ce60b15 Compare April 23, 2024 07:51
@karrui karrui force-pushed the 04-23-fix_remove_erroneous_react_type_pkg_install_on_root branch from 42e4267 to 4bcb262 Compare April 23, 2024 07:51
@karrui karrui force-pushed the 04-23-feat_upgrade_prettier_to_v3 branch from ce60b15 to b608581 Compare April 23, 2024 08:53
@karrui karrui force-pushed the 04-23-fix_remove_erroneous_react_type_pkg_install_on_root branch from 4bcb262 to 2467903 Compare April 23, 2024 08:53
@karrui karrui force-pushed the 04-23-feat_upgrade_prettier_to_v3 branch from b608581 to 7b895f2 Compare April 23, 2024 08:57
@karrui karrui force-pushed the 04-23-fix_remove_erroneous_react_type_pkg_install_on_root branch 2 times, most recently from 149f82d to 1c85806 Compare April 23, 2024 09:33
package.json Show resolved Hide resolved
@karrui karrui force-pushed the 04-23-fix_remove_erroneous_react_type_pkg_install_on_root branch from 1c85806 to 16ac46b Compare April 23, 2024 09:40
@karrui karrui changed the title fix: remove erroneous react type pkg install on root fix: update react type pkg install on root to be same as /frontend Apr 24, 2024
@karrui karrui changed the title fix: update react type pkg install on root to be same as /frontend fix: fix type issues on react-email templates on express server Apr 24, 2024
@karrui karrui force-pushed the 04-23-feat_upgrade_prettier_to_v3 branch from 22d55e6 to 858b012 Compare April 24, 2024 07:57
@karrui karrui force-pushed the 04-23-fix_remove_erroneous_react_type_pkg_install_on_root branch from f5284ae to 0a96443 Compare April 24, 2024 07:57
@karrui karrui requested a review from KenLSM April 29, 2024 12:02
karrui added 2 commits May 2, 2024 12:32
so react-email templates has no type errors when rendering on the server
@karrui karrui force-pushed the 04-23-feat_upgrade_prettier_to_v3 branch from 858b012 to 05f5df4 Compare May 2, 2024 04:34
@karrui karrui force-pushed the 04-23-fix_remove_erroneous_react_type_pkg_install_on_root branch from 0a96443 to 9f75cd7 Compare May 2, 2024 04:34
Copy link
Contributor

@KenLSM KenLSM left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTMM

@karrui karrui mentioned this pull request May 3, 2024
2 tasks
@KenLSM KenLSM merged commit a67ab30 into 04-23-feat_upgrade_prettier_to_v3 May 15, 2024
20 of 22 checks passed
@KenLSM KenLSM deleted the 04-23-fix_remove_erroneous_react_type_pkg_install_on_root branch May 15, 2024 12:34
KenLSM added a commit that referenced this pull request May 20, 2024
* feat: update eslint packages to stable latest versions

not v9 for now since lots of breaking changes
plugin prettier also v4 for now, v5 breaks due to

* fix: add back missed storybook lint import

* fix: remove react-app eslint rules due to incompatibility with eslint

since we are still on CRA4

* fix: add react-hooks eslint plugin since react-app plugin is removed

* feat: add eslint rules to ignore explicit anys in test code

# Conflicts:
#	.eslintrc

* fix: ALL LINT ERRORS DUE TO ANYS

* feat: add looser eslint rules for test files

* feat: upgrade prettier to v3

* fix: formatting changes

* fix: lint for shared

* fix: fix type issues on react-email templates on express server (#7279)

* fix: remove erroneous react type pkg install on root

* fix: add react-email package to root

so react-email templates has no type errors when rendering on the server

* fix: lint issues

* fix: add missing type variable

---------

Co-authored-by: Ken <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants