Skip to content

Update support and care URL with redirection#342

Merged
hendrikebbers merged 3 commits intomainfrom
fix
Apr 13, 2026
Merged

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

Conversation

@Jexsie
Copy link
Copy Markdown
Member

@Jexsie Jexsie commented Apr 9, 2026

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
Copy link
Copy Markdown

netlify bot commented Apr 9, 2026

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
Copy link
Copy Markdown
Member Author

Jexsie commented Apr 9, 2026

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


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