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.

Sign up for GitHub

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

Jump to bottom

Update Clustering In Depth link #317

Closed
srappel opened this issue Oct 16, 2023 · 1 comment · Fixed by #318
Closed

Update Clustering In Depth link #317

srappel opened this issue Oct 16, 2023 · 1 comment · Fixed by #318

Comments

@srappel
Copy link

srappel commented Oct 16, 2023

What is the problem?

The current link in 05-clustering links to https://github.com/OpenRefine/OpenRefine/wiki/Clustering-In-Depth but that page then redirects to https://openrefine.org/docs/technical-reference/clustering-in-depth

Location of problem (optional)

/05-clustering.html
jas58 added a commit that referenced this issue Oct 20, 2023
@jas58
Updated href #317 and made text accessible
2c2e0e1 
Issue #317 identified an outdated link with the new location.  I went on to make the hyperlinked text screenreader accessible by giving it a descriptive, non-url phrase.
@jas58 jas58 mentioned this issue Oct 20, 2023
Merged
@jas58
Copy link
Contributor

jas58 commented Oct 20, 2023

Thank you, Stephen Rappel for catching that. I created a PR #318 . Owen can review it by the end of next week.

jas58 added a commit that referenced this issue Oct 25, 2023
@jas58
Merge pull request #318 from LibraryCarpentry/jas58-patch-1
488ab1d 
Close #317 updated href and made text accessible
github-actions bot pushed a commit that referenced this issue Oct 25, 2023
@actions-user
markdown source builds
5585717 
Auto-generated via {sandpaper}
Source  : 488ab1d
Branch  : main
Author  : Jennifer Stubbs <[email protected]>
Time    : 2023-10-25 15:54:11 +0000
Message : Merge pull request #318 from LibraryCarpentry/jas58-patch-1

Close #317 updated href and made text accessible
github-actions bot pushed a commit that referenced this issue Oct 25, 2023
@actions-user
site deploy
479b4f5 
Auto-generated via {sandpaper}
Source  : 5585717
Branch  : md-outputs
Author  : GitHub Actions <[email protected]>
Time    : 2023-10-25 15:55:45 +0000
Message : markdown source builds

Auto-generated via {sandpaper}
Source  : 488ab1d
Branch  : main
Author  : Jennifer Stubbs <[email protected]>
Time    : 2023-10-25 15:54:11 +0000
Message : Merge pull request #318 from LibraryCarpentry/jas58-patch-1

Close #317 updated href and made text accessible
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Updated href #317 and made text accessible LibraryCarpentry/lc-open-refine
2 participants
@srappel @jas58