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

improved the ui functionality and added new sections #740

Merged
merged 2 commits into from
Jan 2, 2025
Merged

Conversation

muhammad-fiaz
Copy link
Owner

@muhammad-fiaz muhammad-fiaz commented Jan 2, 2025

  • initial update of v2.0.1
  • improved the seo functionality
  • improved UI
  • added new sections

@muhammad-fiaz muhammad-fiaz added documentation Improvements or additions to documentation enhancement New feature or request changes just some changes labels Jan 2, 2025
@muhammad-fiaz muhammad-fiaz self-assigned this Jan 2, 2025
Copy link

vercel bot commented Jan 2, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
portfolio ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 2, 2025 0:53am

@muhammad-fiaz muhammad-fiaz requested a review from Copilot January 2, 2025 12:53
@muhammad-fiaz muhammad-fiaz merged commit 7e12cfb into main Jan 2, 2025
8 checks passed
@muhammad-fiaz muhammad-fiaz deleted the dev branch January 2, 2025 12:53
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 18 out of 33 changed files in this pull request and generated 1 comment.

Files not reviewed (15)
  • package.json: Language not supported
  • public/sitemap-0.xml: Language not supported
  • src/components/content/AboutMe.tsx: Language not supported
  • src/components/ui/Footer.tsx: Language not supported
  • src/components/content/CurrentTimeLineExp.tsx: Evaluated as low risk
  • src/components/ui/ChatBot.tsx: Evaluated as low risk
  • src/components/sections/ProjectsSection.tsx: Evaluated as low risk
  • src/components/content/SupportMe.tsx: Evaluated as low risk
  • src/components/ui/Header.tsx: Evaluated as low risk
  • src/components/sections/BlogSection.tsx: Evaluated as low risk
  • src/components/ui/ExternalLink.tsx: Evaluated as low risk
  • CHANGELOG.md: Evaluated as low risk
  • README.md: Evaluated as low risk
  • src/components/sections/HomeSection.tsx: Evaluated as low risk
  • src/components/content/ContactMe.tsx: Evaluated as low risk
Comments suppressed due to low confidence (2)

src/components/sections/FAQSection.tsx:20

  • The extraction of the main answer text using faq.content.props.children[0] assumes that the first child is always the main answer text, which might not be the case. Ensure that the first child is a string or find a more reliable way to extract the main answer text.
faq.content.props.children[0]

src/app/not-found.tsx:19

  • The URL for the 'Go Back Home' link has been changed from '/' to '/home'. Please confirm if this change was intentional, as it could lead to a broken link.
href="/home"

: children;

return (
<p className='text-base text-gray-600 dark:text-gray-400' dangerouslySetInnerHTML={{ __html: formattedDescription as string }} />
Copy link
Preview

Copilot AI Jan 2, 2025

Choose a reason for hiding this comment

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

Ensure that the children content is sanitized before using dangerouslySetInnerHTML to prevent XSS attacks.

Suggested change
<p className='text-base text-gray-600 dark:text-gray-400' dangerouslySetInnerHTML={{ __html: formattedDescription as string }} />
<p className='text-base text-gray-600 dark:text-gray-400' dangerouslySetInnerHTML={{ __html: DOMPurify.sanitize(formattedDescription as string) }} />

Copilot is powered by AI, so mistakes are possible. Review output carefully before use.

Positive Feedback
Negative Feedback

Provide additional feedback

Please help us improve GitHub Copilot by sharing more details about this comment.

Please select one or more of the options
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changes just some changes documentation Improvements or additions to documentation enhancement New feature or request
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant