Skip to content

Comments

Forms and Data Handling: Change incorrect link in Knowledge Check#29524

Merged
KevinMulhern merged 1 commit intoTheOdinProject:mainfrom
Anshuman-Jha-01:fix/change-link
May 25, 2025
Merged

Forms and Data Handling: Change incorrect link in Knowledge Check#29524
KevinMulhern merged 1 commit intoTheOdinProject:mainfrom
Anshuman-Jha-01:fix/change-link

Conversation

@Anshuman-Jha-01
Copy link
Contributor

Because

This PR updates an incorrect link under Knowledge Check section of Forms and Data Handling topic.

This PR

  • Fixes the incorrect link which leads to a page about SQL Injection in the final question in the Knowledge Check section to a link that leads to a section about escaping HTML characters.

Issue

Closes #29515

Pull Request Requirements

  • I have thoroughly read and understand The Odin Project curriculum contributing guide
  • The title of this PR follows the location of change: brief description of change format, e.g. Intro to HTML and CSS lesson: Fix link text
  • The Because section summarizes the reason for this PR
  • The This PR section has a bullet point list describing the changes in this PR
  • If this PR addresses an open issue, it is linked in the Issue section
  • If any lesson files are included in this PR, they have been previewed with the Markdown preview tool to ensure it is formatted correctly
  • If any lesson files are included in this PR, they follow the Layout Style Guide

@github-actions github-actions bot added the Content: NodeJS Involves the NodeJS course label Mar 17, 2025
@wise-king-sullyman wise-king-sullyman added the Status: Do Not Merge This PR should not be merged until further notice label Mar 18, 2025
Copy link
Member

@KevinMulhern KevinMulhern left a comment

Choose a reason for hiding this comment

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

@KevinMulhern KevinMulhern merged commit a49a0ab into TheOdinProject:main May 25, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Content: NodeJS Involves the NodeJS course Status: Do Not Merge This PR should not be merged until further notice

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Forms and Data Handling: Update link in Knowledge Check

3 participants