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

Waku integration #287

Draft
wants to merge 134 commits into
base: release
Choose a base branch
from
Draft

Waku integration #287

wants to merge 134 commits into from

Conversation

pmelab
Copy link
Contributor

@pmelab pmelab commented Jun 23, 2024

Description of changes

Swap out Gatsby for Waku.

Motivation and context

Gatsby is not progressing and barely maintained. There is no hope that our current issues will get resolved any time soon.

Remaining issues

How has this been tested?

  • Manually
  • Unit tests
  • Integration tests

@pmelab pmelab force-pushed the lagoon-waku branch 3 times, most recently from 1ef4712 to 5c76a10 Compare June 25, 2024 14:08
@pmelab pmelab changed the title Lagoon waku Waku integration Jun 27, 2024
"serve": "netlify dev --cwd=. --dir=public --port=8000",
"serve:proxy": "mitmproxy -p 8000 --mode reverse:http://localhost:8002/",
"serve:proxied": "netlify dev --cwd=. --dir=dist/public --port=8002",
"serve": "netlify dev --cwd=. --dir=dist/public --port=8000",
"dev": "pnpm clean && publisher",
Copy link
Contributor

Choose a reason for hiding this comment

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

The clean command is missing here I guess - Phillip responded that it has not cache right now, so there is not point to clear, so perhaps remove it?

<ErrorBoundary
onError={(error) => {
if ((error as any).statusCode === 404) {
window.location.reload();
Copy link
Member

Choose a reason for hiding this comment

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

Aha, so this is the reason of https://waku.amazeelabs.dev/en/content-hub craziness 😅

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, this has to become more solid.

if (result.state === 'success') {
const rawTranslations = result.data.stringTranslations || [];
const translations = {
...translationsMap(
Copy link
Member

Choose a reason for hiding this comment

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

Just FYI. There is a big performance difference between [...arr1, ...arr2] and arr1.concat(arr2). Can be essential on big arrays.

if (isLocale(prefix)) {
setCurrentLocale(prefix);
}
}, [setCurrentLocale]);
Copy link
Member

Choose a reason for hiding this comment

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

Just a guess: should window.location.pathname be passed as a dependency as well?

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.

4 participants