Skip to content

Fix react error 310#3

Open
creatorcluster wants to merge 27 commits intoYxmura:mainfrom
creatorcluster:fix-react-error-310
Open

Fix react error 310#3
creatorcluster wants to merge 27 commits intoYxmura:mainfrom
creatorcluster:fix-react-error-310

Conversation

@creatorcluster
Copy link
Contributor

No description provided.

creatorcluster and others added 27 commits July 9, 2025 16:42
might work
The React error #310 was caused by a conditional call to the `useDiscordPopup` hook in `App.tsx`. This change moves the hook to before the conditional return statement, ensuring that it is called on every render and resolving the error.
The React error #310 was caused by the i18n initialization logic in `App.tsx`, which caused the component to render a different component tree on the initial render. This change moves the i18n initialization to `main.tsx` to ensure that the `App` component is only rendered after the i18n library is fully initialized.
This commit resolves two issues:

1.  **React Error #310:** This error was caused by conditional rendering of hooks. I moved the i18n initialization from `App.tsx` to `main.tsx` to ensure that the `App` component is only rendered after the i18n library is fully initialized.

2.  **i18n Translation Issues:** After moving the i18n initialization, the translation keys were being displayed instead of the translated text. This was because the `i18n` instance was not being properly passed to the `I18nextProvider`. I moved the `I18nextProvider` to `main.tsx` and the `i18n` instance is now correctly passed to it.
This commit resolves all i18n and react errors by correctly initializing the i18n instance in `main.tsx` and removing the redundant i18n logic from `App.tsx`.
This commit removes the i18n feature from the application and replaces all translation keys with their English text. This simplifies the codebase and removes the complexity of managing translations.
This commit removes the i18n feature from the application and replaces all translation keys with their English text. This simplifies the codebase and removes the complexity of managing translations.
This commit removes the i18n feature from the application and replaces all translation keys with their English text. This simplifies the codebase and removes the complexity of managing translations.
Yxmura pushed a commit that referenced this pull request Sep 28, 2025
…nd-organization-6fd4

Fix website labels and organization
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.

2 participants