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

Update incorrect link in address and change language patterns #3210

Merged
merged 3 commits into from
May 22, 2024

Conversation

precious-onyenaucheya-ons
Copy link
Contributor

@precious-onyenaucheya-ons precious-onyenaucheya-ons commented May 17, 2024

What is the context of this PR?

Fixes #3207 and #3208

This PR updates the links in the address and change language pattern examples

How to review this PR

  • Check that all links in the pattern works in the DS and SM projects

Checklist

This needs to be completed by the person raising the PR.

  • I have selected the correct Assignee
  • I have linked the correct Issue

@precious-onyenaucheya-ons precious-onyenaucheya-ons added the Bug Something isn't working label May 17, 2024
@precious-onyenaucheya-ons precious-onyenaucheya-ons requested a review from a team May 17, 2024 06:41
Copy link

netlify bot commented May 17, 2024

Deploy Preview for ons-design-system-preview ready!

Name Link
🔨 Latest commit 51a82bf
🔍 Latest deploy log https://app.netlify.com/sites/ons-design-system-preview/deploys/664b749bbd01f20009ca4cc5
😎 Deploy Preview https://deploy-preview-3210--ons-design-system-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

@precious-onyenaucheya-ons precious-onyenaucheya-ons linked an issue May 17, 2024 that may be closed by this pull request
@rmccar
Copy link
Contributor

rmccar commented May 17, 2024

We need to make the PR title and description a bit more descriptive and a bit more explanation of how to review.

I haven't tested it in the service manual but the change seems to make no difference in the DS and the content of the Welsh example in the DS is all in Welsh already so is this a service manual issue?

Copy link
Contributor

@alessioventuriniAND alessioventuriniAND left a comment

Choose a reason for hiding this comment

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

Can you also please remove the +++ in the change language files? These cause issues in the service manual as the example don't display.

@precious-onyenaucheya-ons precious-onyenaucheya-ons changed the title update link update incorrect link in address and change language pattern May 17, 2024
@rmccar rmccar changed the title update incorrect link in address and change language pattern Update incorrect link in address and change language patterns May 17, 2024
@precious-onyenaucheya-ons precious-onyenaucheya-ons merged commit 5e4df81 into main May 22, 2024
9 checks passed
@precious-onyenaucheya-ons precious-onyenaucheya-ons deleted the fix/incorrect-links-in-pattern branch May 22, 2024 09:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

The link in address pattern is wrong Change language pattern page broken in service manual
5 participants