Skip to content

Refactor code structure for improved readability and maintainability#4

Merged
HiVladius merged 1 commit intomainfrom
feature/projects-page
May 12, 2025
Merged

Refactor code structure for improved readability and maintainability#4
HiVladius merged 1 commit intomainfrom
feature/projects-page

Conversation

@HiVladius
Copy link
Owner

This pull request introduces a variety of changes, including dependency updates, bug fixes, enhancements to the ProjectsSection component, and improvements to internationalization and translations. Below is a summary of the most important changes grouped by theme.

Dependency Updates

  • Updated react and react-dom dependencies in package.json from version 18.3.1 to 19.1.0 to ensure compatibility with the latest React features and improvements.

Bug Fixes and Code Improvements

  • Updated the setRepos function in ProjectsSection to filter out duplicate repositories by their id, ensuring unique entries in the repository list.
  • Fixed the key prop for repository items and their topics in ProjectsSection to avoid potential React rendering issues by using unique identifiers. [1] [2]
  • Replaced a conditional rendering approach for the loading spinner in ProjectsSection with a ternary operator for better readability.

Internationalization Enhancements

  • Updated the title prop in the AboutSection component to use a translated string (t("hobbies-section.my hobbies")) for improved localization.
  • Added a new translation entry (hobbies-1.2) in the English translation file for better content coverage.

Other Changes

  • Made the firstLine prop in ModalProfileProps optional to provide greater flexibility in component usage.
  • Increased the loading timeout in AppWrapper from 2500ms to 3000ms for a smoother user experience during initialization.

@vercel
Copy link

vercel bot commented May 12, 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 May 12, 2025 7:23pm

@HiVladius HiVladius merged commit 20be012 into main May 12, 2025
3 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.

1 participant