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

Mohtady on board #2919

Merged
merged 3 commits into from
Jul 31, 2023
Merged

Conversation

MohtadyDeveloper
Copy link
Contributor

Adding My Avatar

@zero-to-mastery-bot
Copy link

🙏 Thanks for your pull request @MohtadyDeveloper, The team will now review and merge this request. In the mean time why not check out some of the other opensource projects available, contributions are greatly appreciated!

Some of the most popular are


PR Statistics

#️⃣ PR Number: Line Additions: 🗑️ Line Deletions:
2919 16 0
📑 Files Changed: Repo Stars: 🔱 Total Forks:
1 953 2636

@MohtadyDeveloper
Copy link
Contributor Author

When I can see my self here https://alumni-board.netlify.app/

@l-white
Copy link
Contributor

l-white commented Jul 29, 2023

@MohtadyDeveloper Could you replace the website with "" if you don't have one? I see the placeholder text in the commit.

@zero-to-mastery-bot
Copy link

🙏 Thanks for your pull request @MohtadyDeveloper, The team will now review and merge this request. In the mean time why not check out some of the other opensource projects available, contributions are greatly appreciated!

Some of the most popular are


PR Statistics

#️⃣ PR Number: Line Additions: 🗑️ Line Deletions:
2919 16 0
📑 Files Changed: Repo Stars: 🔱 Total Forks:
1 954 2636

@MohtadyDeveloper
Copy link
Contributor Author

I replaced the website with ""

@MohtadyDeveloper
Copy link
Contributor Author

{
"name": "Mohtady Ekmail",
"img": "https://github.com/avatars/u/128006116?v=4",
"email": "Mohtady.developer@gmail.com",
"links": {
"website": "",
"linkedin": "https://www.linkedin.com/in/mohtady-ekmail-35ab08b",
"github": "https://github.com/MohtadyDeveloper"
},
"jobTitle": "Full Stack Developer",
"location": {
"city": "Roseville",
"state": "Sacramento, California",
"country": "USA"
}
}

@MohtadyDeveloper
Copy link
Contributor Author

correct?

@l-white
Copy link
Contributor

l-white commented Jul 30, 2023

@MohtadyDeveloper This is correct but you need to push the change to the code as you did for the other changes. Once you commit that change we can merge your pull request

@zero-to-mastery-bot
Copy link

🙏 Thanks for your pull request @MohtadyDeveloper, The team will now review and merge this request. In the mean time why not check out some of the other opensource projects available, contributions are greatly appreciated!

Some of the most popular are


PR Statistics

#️⃣ PR Number: Line Additions: 🗑️ Line Deletions:
2919 16 0
📑 Files Changed: Repo Stars: 🔱 Total Forks:
1 954 2638

@l-white l-white merged commit 7200027 into zero-to-mastery:master Jul 31, 2023
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants