Skip to content

Conversation

@bilalmirza74
Copy link
Contributor

Summary

3D Portfolio Website using Three.js with React

Screenshot

image

Live Project Link

https://portfoliowebme.netlify.app

@bilalmirza74
Copy link
Contributor Author

Please don't forget to add hacktoberfest, hacksquad

Copy link

@check-bot check-bot left a comment

Choose a reason for hiding this comment

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

looking cool🚀

Copy link
Collaborator

@PBJI PBJI left a comment

Choose a reason for hiding this comment

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

Hi @bilalmirza74, your repository requires some changes before it can be merged:


❌ RENAMING: You should considering renaming your root folder into 3DPorffolio folder 3D-Portfolio folder.


❌ DIRECTORY: Your project files, including the README.md should be inside another folder called bilalmirza74, so ideally your pull request changes would look something like this:

javascript-mini-projects
|           
-------- 3DPorffolio
           |
            ----- bilalmirza74 
                   |
                    -------- **Your project files and folders**
                   |
                    --------- README.md (optional yet preferred)

❓ REFERENCES:

  1. Refer to How to raise the PR for more information

Once you make these necessary changes, please revert back with a comment mentioning @PBJI so that I get notified.

Thank you. 😄

@PBJI PBJI added the changeRequested The requested changes should be adhered to before further review or approval. label Oct 8, 2023
@bilalmirza74
Copy link
Contributor Author

Done bro @PBJI

@PBJI
Copy link
Collaborator

PBJI commented Oct 8, 2023

@bilalmirza74 RENAMING Root directory, remove the - and use pascal case and you are good to go.

@bilalmirza74
Copy link
Contributor Author

bilalmirza74 commented Oct 8, 2023

@PBJI
Sorry for wrong commit Now Done

@bilalmirza74 bilalmirza74 requested a review from PBJI October 8, 2023 19:37
@bilalmirza74
Copy link
Contributor Author

@PBJI
Please add hacktoberfest, hacksquad

@PBJI PBJI merged commit 28f2ce3 into thinkswell:master Oct 8, 2023
@bilalmirza74 bilalmirza74 mentioned this pull request Oct 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changeRequested The requested changes should be adhered to before further review or approval.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants