Skip to content

Update support and care URL with redirection#342

Merged
hendrikebbers merged 3 commits into
mainfrom
fix
Apr 13, 2026
Merged

Update support and care URL with redirection#342
hendrikebbers merged 3 commits into
mainfrom
fix

Conversation

@Jexsie

@Jexsie Jexsie commented Apr 9, 2026

Copy link
Copy Markdown
Member

Description

fixes #339 #339 #338 #341

Type of Change

  • 🐛 Bug fix
  • ✨ New feature
  • 📝 Documentation update
  • 🎨 Style/formatting
  • ♻️ Refactor
  • 🔧 Configuration
  • ⚡ Performance
  • 🧪 Tests
  • 🔐 Security

Related Issue(s)

Closes #ISSUE_NUMBER

Changes Made

  • Change 1
  • Change 2
  • Change 3

How to Test

  1. Step 1
  2. Step 2
  3. Verify the expected behavior

Checklist

  • Code follows project style guidelines
  • Self-review conducted
  • Comments added for complex logic
  • Documentation updated (if needed)
  • No new warnings generated
  • Tests added/updated (if applicable)
  • All tests passing locally

Jexsie added 2 commits April 9, 2026 10:51
Signed-off-by: Jessie Ssebuliba <jessiessebuliba@gmail.com>
Signed-off-by: Jessie Ssebuliba <jessiessebuliba@gmail.com>
@netlify

netlify Bot commented Apr 9, 2026

Copy link
Copy Markdown

Deploy Preview for open-elements ready!

Name Link
🔨 Latest commit 68375aa
🔍 Latest deploy log https://app.netlify.com/projects/open-elements/deploys/69dc9552058a0200081064f2
😎 Deploy Preview https://deploy-preview-342--open-elements.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 67
Accessibility: 97
Best Practices: 92
SEO: 100
PWA: 60
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

@Jexsie

Jexsie commented Apr 9, 2026

Copy link
Copy Markdown
Member Author

There is no en version for the newsletter, that's why it was broken. I have added a temporary redirection to the de version

Comment thread src/app/[locale]/newsletter/page.tsx Outdated

if (locale !== 'de') {
notFound();
redirect({ href: '/newsletter', locale: 'de' });

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

No, we should not "by default" switch to German.

@danielmarv

Copy link
Copy Markdown
Member

@Jexsie
the test are not passing

Signed-off-by: Jessie Ssebuliba <jessiessebuliba@gmail.com>
@hendrikebbers
hendrikebbers merged commit 5548dcd into main Apr 13, 2026
9 of 10 checks passed
@Jexsie
Jexsie deleted the fix branch April 13, 2026 07:06
@Jexsie Jexsie mentioned this pull request Apr 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Newsletter & Archive in Footer

3 participants