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 react-fullpage library and documentation #60

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ThulaniMthembu
Copy link

  • Enhance package.json with new metadata and dependencies
  • Resolve React version conflicts (support React 17 and 18)
  • Fix invalid hook call errors in Fullpage component
  • Address npm audit vulnerabilities in development dependencies
  • Improve TypeScript support and resolve type errors
  • Boost test coverage to 100% pass rate
  • Create README_ENHANCED.txt with detailed updates and test results
  • Update main README.md with links to enhanced documentation
  • Rename branch to avoid conflicts with original master

- Enhance package.json with new metadata and dependencies
- Resolve React version conflicts (support React 17 and 18)
- Fix invalid hook call errors in Fullpage component
- Address npm audit vulnerabilities in development dependencies
- Improve TypeScript support and resolve type errors
- Boost test coverage to 100% pass rate
- Create README_ENHANCED.txt with detailed updates and test results
- Update main README.md with links to enhanced documentation
- Rename branch to avoid conflicts with original master
@ThulaniMthembu
Copy link
Author

Hi Thierry, I've submitted a pull request with several updates and improvements to the react-fullpage library. These changes include enhanced documentation, bug fixes, and improved compatibility with React 17 and 18. I've also addressed some npm audit vulnerabilities and improved our test coverage.

The detailed changes can be found in the pull request description. I welcome any feedback or suggestions for further improvements. Thank you for considering these updates!

@thierryc
Copy link
Owner

Thank you so much for your amazing contribution! 🎉 I’ll take a glance through everything next week and will follow up with a more detailed review after that. I’ll keep you posted on how things go.

Your hard work is really appreciated 🙌

Thanks again!

@thierryc
Copy link
Owner

thierryc commented Oct 23, 2024

Hi @ThulaniMthembu,

I wanted to bring up a small but important detail regarding the package name.
To avoid potential conflicts with other projects in the NPM repository, the package should be named @ap.cx/react-fullpage rather than just react-fullpage.

This namespace helps ensure that the package remains unique and avoids issues with existing packages. Could you please update the package name accordingly?

Additionally, I will need to update the demo page on the gh-pages branch before merging. Sorry for the delay, but I’ll try to get that done this week.

Could you provide a working example or a link to a demo showcasing the changes? It would help in understanding the implementation better and ensure everything works as expected in real scenarios.

Appreciate it!

Thanks again for your work!

Best,
Thierry

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.

2 participants