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

Added Star Meshery Repository button #319

Merged
merged 5 commits into from
Aug 22, 2023

Conversation

vednp
Copy link

@vednp vednp commented Jul 17, 2023

Description

This PR fixes #299 Added a button on the homepage under the Nighthawk and Meshery section to redirect the user to the Meshery Repository

Notes for Reviewers

Signed commits

  • Yes, I signed my commits.

@welcome
Copy link

welcome bot commented Jul 17, 2023

Yay, your first pull request! 👍 A contributor will be by to give feedback soon. In the meantime, please review the Layer5 Community Welcome Guide and sure to join the community Slack.
Be sure to double-check that you have signed your commits. Here are instructions for making signing an implicit activity while peforming a commit.

@netlify
Copy link

netlify bot commented Jul 17, 2023

Website preview ready!

Name Link
🔨 Latest commit 34896cd
🔍 Latest deploy log https://app.netlify.com/sites/getnighthawk/deploys/64b6807a6f715600082f496e
😎 Deploy Preview https://deploy-preview-319--getnighthawk.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Ved Pahune added 2 commits July 18, 2023 01:42
docs/index.html Outdated
Comment on lines 88 to 90
<div class="star-meshery"><a class="star-meshery-button" href="https://github.com/meshery/meshery" target="_blank"
"> ⭐️ Star Meshery Repository </a>
</div>
Copy link
Member

@Chadha93 Chadha93 Jul 18, 2023

Choose a reason for hiding this comment

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

@vednp Thanks for the PR, let's not create a button like this.
Please refer to the https://meshery.io/ site, we need to replicate it.

Copy link
Author

Choose a reason for hiding this comment

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

Sure, Is the position correct or should I add it in the hero section like in meshery's website?

Copy link
Member

Choose a reason for hiding this comment

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

Yes, it's correct.

Ved Pahune added 2 commits July 18, 2023 17:30
@vednp vednp requested a review from Chadha93 July 18, 2023 12:08
@vednp
Copy link
Author

vednp commented Jul 19, 2023

@leecalcote @Chadha93 @Shivam-AfA can you all review this?

@leecalcote
Copy link
Member

leecalcote commented Jul 19, 2023

Having the Meshery repo star button and count is great.

We also need one for Nighthawk, though, too, which should point to this repo (the one that we're commenting on right now), @Chadha93

@vednp
Copy link
Author

vednp commented Jul 20, 2023

@leecalcote Should I work on it? Btw we have a link directing to this repo on the top right corner but it's not visible on mobile view so we can also add a hamburger menu for lesser screen dimensions. Your opinions?

@Chadha93
Copy link
Member

@vednp You can proceed with Lee's suggestion.

@l5io
Copy link
Contributor

l5io commented Aug 22, 2023

@Chadha93 please draw this to a close.

@leecalcote
Copy link
Member

Thanks!

@leecalcote leecalcote merged commit 46e2785 into layer5io:master Aug 22, 2023
@welcome
Copy link

welcome bot commented Aug 22, 2023

Thanks for your contribution to the Layer5 community! 🎉

Congrats!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Add "Star Meshery Repository ⭐ " button
4 participants