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

feat: add note for boundary desktop client #2672

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

LeahMarieBush
Copy link
Contributor

@LeahMarieBush LeahMarieBush commented Jan 21, 2025

πŸ”— Relevant links

πŸ—’οΈ What

This PR adds a note to the bottom of the 'Desktop Client' section for the boundary install page. The note links to the previous versions of the desktop client.

🀷 Why

Previous versions of boundary are not always compatible with the latest version of the desktop client. Linking to that page will allow users to download the version compatible with their boundary version.

πŸ“Έ Design Screenshots

Screenshot 2025-01-21 at 3 56 55β€―PM Screenshot 2025-01-21 at 3 57 03β€―PM

πŸ§ͺ Testing

  1. Go to preview link
  2. See the note at the bottom of that section and verify that it matches design in Asana task
  3. Click the link and make sure it takes you to the install page

Copy link

vercel bot commented Jan 21, 2025

The latest updates on your projects. Learn more about Vercel for Git β†—οΈŽ

Name Status Preview Comments Updated (UTC)
dev-portal βœ… Ready (Inspect) Visit Preview πŸ’¬ Add feedback Jan 21, 2025 9:39pm

Copy link

github-actions bot commented Jan 21, 2025

πŸ“¦ Next.js Bundle Analysis

This analysis was generated by the next.js bundle analysis action πŸ€–

This PR introduced no changes to the javascript bundle πŸ™Œ

@LeahMarieBush LeahMarieBush marked this pull request as ready for review January 21, 2025 21:57
@LeahMarieBush LeahMarieBush requested review from a team, rmainwork and emilypersson1 and removed request for a team January 21, 2025 21:57
Copy link

@emilypersson1 emilypersson1 left a comment

Choose a reason for hiding this comment

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

looks great, thanks Leah!

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