Skip to content

fix: updated 404 link in the Footer.test.tsx file - i167#169

Merged
DianaLease merged 3 commits intoaccordproject:mainfrom
Protik49:Protik49/i167/update-404-link
Mar 11, 2025
Merged

fix: updated 404 link in the Footer.test.tsx file - i167#169
DianaLease merged 3 commits intoaccordproject:mainfrom
Protik49:Protik49/i167/update-404-link

Conversation

@Protik49
Copy link
Contributor

@Protik49 Protik49 commented Mar 5, 2025

Closes #167

Summary of the Pull Request

This pull request addresses issue #167 by updating the "VS Code Extension" link in the footer of the Accord Project Playground. The previous link led to a 404 error page; it has now been redirected to the correct Accord Project (Concerto) extension link.

Changes

  • Updated the "VS Code Extension" link in the footer to point to the Accord Project (Concerto) extension.

  • Modified the Footer.test.tsx file to reflect the updated link, ensuring that the test cases accurately validate the new URL.

Screenshots or Video

  • Before
befor
  • After
after

Author Checklist

  • Ensure you provide a DCO sign-off for your commits using the --signoff option of git commit.
  • Vital features and changes captured in unit and/or integration tests
  • Commits messages follow AP format
  • Extend the documentation, if necessary
  • Merging to main from fork:branchname

Signed-off-by: Protik49 <protikprotik49@gmail.com>
@Protik49 Protik49 requested a review from a team as a code owner March 5, 2025 09:40
@netlify
Copy link

netlify bot commented Mar 5, 2025

Deploy Preview for ap-template-playground ready!

Name Link
🔨 Latest commit 9753ff8
🔍 Latest deploy log https://app.netlify.com/sites/ap-template-playground/deploys/67c9e019e4cd4d0007dba097
😎 Deploy Preview https://deploy-preview-169--ap-template-playground.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.

Copy link
Member

@DianaLease DianaLease left a comment

Choose a reason for hiding this comment

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

Thank you for the fix!

Copy link
Member

@DianaLease DianaLease left a comment

Choose a reason for hiding this comment

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

Sorry, I only see the change in the test file though. Can you update the source file as well?

@DianaLease DianaLease merged commit 70bd7aa into accordproject:main Mar 11, 2025
7 checks passed
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.

VS Code Extension Link Returns 404

2 participants