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

Broken references in Generic Sensor API #443

Closed
4 tasks
dontcallmedom-bot opened this issue Dec 7, 2022 · 2 comments · Fixed by #446 or w3c/strudy#252
Closed
4 tasks

Broken references in Generic Sensor API #443

dontcallmedom-bot opened this issue Dec 7, 2022 · 2 comments · Fixed by #446 or w3c/strudy#252

Comments

@dontcallmedom-bot
Copy link

While crawling Generic Sensor API, the following links to other specifications were detected as pointing to non-existing anchors:

This issue was detected and reported semi-automatically by Strudy based on data collected in webref.
@anssiko
Copy link
Member

anssiko commented Dec 7, 2022

I believe @rakuco will be addressing these and possibly also other WebDriver-related clarifications and fixes.

@rakuco
Copy link
Member

rakuco commented Dec 7, 2022

I've got a fix here but can't get Bikeshed to find the WebDriver definitions I am referencing -- I've filed speced/bikeshed#2416

rakuco added a commit to rakuco/sensors that referenced this issue Dec 8, 2022
@rakuco
Fix broken references to WebDriver definitions.
9c9e6eb 
We were hardcoding some dfn URLs `anchors` section that have changed in the
WebDriver spec, resulting in broken links.

Several of those actually became exported definitions in w3c/webdriver#1639,
so we can simply rely on Bikeshed autolinking working for them. In these
cases, remove the hardcoded dfns and adjust the wording in the spec (e.g.
"WebDriver error" -> "error").

Additionally, use the "WebDriver-2" shortname to refer to the spec, as it
points to the current version which includes the exported definitions that
we need.

Finally, also adapt to w3c/webdriver#1345 which renamed the "Handling
Errors" section to "Errors".

Fixes w3c#443.
@rakuco rakuco mentioned this issue Dec 8, 2022
Merged
@anssiko anssiko closed this as completed in 0829d7b Dec 8, 2022
@github-actions github-actions bot mentioned this issue Dec 14, 2022
Merged
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
3 participants
@rakuco @anssiko @dontcallmedom-bot