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

feat: update react-app with latest romcal calendar & features #13

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

emagnier
Copy link
Contributor

@emagnier emagnier commented Oct 25, 2024

Update of the React app site, which also serves as the base for romcal.js.org.

@@ -5,7 +5,7 @@
"type": "module",
"scripts": {
"build": "tsc && NODE_NO_WARNINGS=1 vite build",
"build:dev": "VITE_DAY_VARIANT=developer npm run build",
"build:dev": "NODE_OPTIONS='--max-old-space-size=8192' VITE_DAY_VARIANT=developer npm run build",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

With more and more calendars and data, I had to increase the memory here, otherwise I couldn't build the site anymore.

"@tanstack/react-query": "^5.56.2",
"classnames": "^2.5.1",
"date-fns": "^4.1.0",
"flatted": "^3.3.1",
"i18next": "^23.15.2",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This has been new since converting the Romcal codebase into a monorepo. I'm not sure if it's a bug or an improvement... Previously, i18next was bundled with Romcal's source code, but now it has to be added explicitly to use Romcal.

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.

1 participant