Skip to content

Update support and care URL with redirection#342

Open
Jexsie wants to merge 3 commits intomainfrom
fix
Open

Update support and care URL with redirection#342
Jexsie wants to merge 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 7d5beac
🔍 Latest deploy log https://app.netlify.com/projects/open-elements/deploys/69d75f9af6d9ed000882356c
😎 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: 72
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.

Signed-off-by: Jessie Ssebuliba <jessiessebuliba@gmail.com>
@danielmarv
Copy link
Copy Markdown
Member

@Jexsie
the test are not passing

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