Skip to content
This repository has been archived by the owner on Aug 21, 2024. It is now read-only.

Changing the "Getting Started" section cards #563

Merged

Conversation

ElliotFriend
Copy link
Contributor

This commit does a couple things:

  1. Fixes an old link for the "deploy to a local network" card that was moved some time ago. We were receiving a few 404 errors for that old URL.
  2. Changes the cards to be more in-line with the current structure of the "Getting Started" section of docs on our site.

I've removed the link to "freighter wallet" since freighter is covered in the "create an app" page, although I'm not entirely sure if that or "deploy to a local network" is more useful to have here. Would love some thoughts from others.

This commit does a couple things:

1. Fixes an old link for the "deploy to a local network" card that
   was moved some time ago. We were receiving a few 404 errors for
   that old URL.
2. Changes the cards to be more in-line with the current structure
   of the "Getting Started" section of docs on our site.

I've removed the link to "freighter wallet" since freighter is
covered in the "create an app" page, although I'm not entirely
sure if that or "deploy to a local network" is more useful to have
here. Would love some thoughts from others.
@stellar-jenkins
Copy link

@briwylde08
Copy link
Contributor

I think deploy to a local network makes more sense... since Freighter is in the application section.

Copy link
Contributor

@briwylde08 briwylde08 left a comment

Choose a reason for hiding this comment

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

LGTM!

@ElliotFriend ElliotFriend merged commit 6a0ef9c into stellar-deprecated:main Sep 15, 2023
1 check passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants