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

CLICK TO SEE PORTFOLIO-->https://amporfolio.surge.sh/index #145

Closed
wants to merge 1 commit into from

Conversation

aditimahabole
Copy link

https://amporfolio.surge.sh/index

Portfolio Author

Portfolio Link?

GitHub Link:

Tech Stack

@github-actions
Copy link

github-actions bot commented Oct 3, 2022

Hello @aditimahabole, 🔥 thanks for raising a pull request in this project. Now, sit back and drink some coffee while we review this.

Copy link
Contributor

@piyush-multiplexer piyush-multiplexer left a comment

Choose a reason for hiding this comment

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

Hi @aditimahabole, thanks for your contribution.
But you need to provide portfolio details in the README file, please check the contribution guidelines.

@Evavic44
Copy link
Owner

Evavic44 commented Oct 4, 2022

Hello, @aditimahabole thanks for contributing to this project, but as @piyush-multiplexer said, portfolio should be added to the Readme file inside the table.

Here's an open issue you can check out #104.

I'll be closing this pull request for now.

Thanks for your PR.

@Evavic44 Evavic44 closed this Oct 4, 2022
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.

3 participants