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

Whimsy failed to find Thanks link #191

Open
henrysun007 opened this issue Apr 4, 2023 · 4 comments
Open

Whimsy failed to find Thanks link #191

henrysun007 opened this issue Apr 4, 2023 · 4 comments

Comments

@henrysun007
Copy link

The check result in https://whimsy.apache.org/pods/project/teaclave shows no link to Thanks. However, we have added the Thanks link in http://teaclave.incubator.apache.org/ at the top right of the web page. The source code is in https://github.com/henrysun007/incubator-teaclave-website.

Does anyone know the reason?

@sebbASF
Copy link
Contributor

sebbASF commented Apr 4, 2023

The link body includes the following:

(opens new window)

This is not expected to be present as part of the link text.

Given that it is not visible, I wonder what it is doing?

@henrysun007
Copy link
Author

Can opens new window be supported in the future? I think it is acceptable for a link to open a new window.

@ShaneCurcuru
Copy link
Contributor

The purpose of this check is to ensure that at least "Thanks", a common English word, appears in the link text, and it goes to the correct page. There's no policy reason to forbid the extra text, it just requires someone to update and test a new regex for the actual check code:

https://github.com/apache/whimsy/blob/master/lib/whimsy/sitestandards.rb#L71

@sebbASF
Copy link
Contributor

sebbASF commented Apr 4, 2023

However that is not what the policy currently says.

As to opening in a separate window, that is not an issue.
The text does not actually appear as far as I can tell, and there is anyway an icon which indicates that a new window will be used

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

No branches or pull requests

3 participants