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 README.md #8

Merged
merged 1 commit into from
Jun 23, 2023
Merged

Update README.md #8

merged 1 commit into from
Jun 23, 2023

Conversation

the-faizmohammad
Copy link
Owner

Pull Request Summary

This pull request proposes the deployment of the website using GitHub Pages, along with necessary updates to ensure the smooth functioning of the online version of the portfolio. Additionally, it suggests an enhancement to the repository's README file by incorporating a link to the deployed online version in Live Demo Section of README file.

The following changes have been made:

  1. Deployment of Website: The website has been deployed using GitHub Pages, allowing for easy accessibility and visibility to users.

  2. Functionality Testing: Extensive testing has been performed on the online version of the portfolio to verify that all features and pages are functioning as intended. This ensures a seamless user experience and confirms that the deployed website aligns with the original design and functionality.

  3. README Update: The README file in the repository has been modified to include a direct link to the online version of the portfolio. This enhancement enables visitors and contributors to easily access the deployed website from the repository.

These changes contribute to improved accessibility, convenience, and collaboration for users and developers. By deploying the website and providing a link in the README, it enhances the overall user experience and allows for efficient feedback and contributions.

Please review this pull request and provide any necessary feedback or suggestions for further improvements.

Copy link

@LionRouge1 LionRouge1 left a comment

Choose a reason for hiding this comment

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

Hi @the-faizmohammad,

STATUS: APPROVED! 🥇 💯

Your project is complete! There is nothing else to say other than... it's time to merge it :shipit:

done

Good luck! 🎉

Feel free to leave any questions or comments in the PR thread if something is not 100% clear.

@the-faizmohammad the-faizmohammad merged commit 268bac7 into main Jun 23, 2023
3 of 4 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.

2 participants