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

fix link #7

Closed
wants to merge 1 commit into from
Closed

fix link #7

wants to merge 1 commit into from

Conversation

PanMan
Copy link

@PanMan PanMan commented Dec 7, 2021

No description provided.

Copy link
Member

@tombruijn tombruijn left a comment

Choose a reason for hiding this comment

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

Hi @PanMan, Thanks for the PR! Can you update the PR with the suggestion I made?

@@ -71,7 +71,7 @@ The examples can always be started using the `npm start`/`yarn start` commands.

## Setup

1. Create an application on [AppSignal.com][appsignal-website].
1. Create an application on [appsignal-website](https://AppSignal.com).
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
1. Create an application on [appsignal-website](https://AppSignal.com).
1. Create an application on [AppSignal.com][appsignal].

I see there's a link alias called appsignal that we can use here instead.

@backlog-helper
Copy link

While performing the daily checks some issues were found with this Pull Request.

  • This Pull Request has not had any activity in 14 days. Please provide a status update if it is still relevant. Closed it if it is no longer relevant. Or move it to another column if it's blocked or requires another look at it. - (More info)

New issue guide | Backlog management | Rules | Feedback

tombruijn added a commit that referenced this pull request Jan 3, 2022
As reported in #7.
Closes #7.

Co-authored-by: PanMan <github@panman.nl>
@tombruijn tombruijn closed this Jan 3, 2022
@tombruijn
Copy link
Member

Merged in 26e564c.

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.

2 participants