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

Update index.html #1368

Merged
merged 1 commit into from
Oct 20, 2022
Merged

Update index.html #1368

merged 1 commit into from
Oct 20, 2022

Conversation

Iza82
Copy link
Contributor

@Iza82 Iza82 commented Oct 6, 2022

Few minor changes within the text

@situx
Copy link
Collaborator

situx commented Oct 7, 2022

Everything seems fine except for this part Geographic information/Geomatics.
The tag should be a hyperlink pointing to an internet address, but no internet address is given. @Iza82

@andrea-perego
Copy link
Contributor

It seems there are a few things to be fixed - ReSpec raises 4 warnings and 2 errors:

https://raw.githack.com/w3c/sdw/iza-review/bp/index.html

About this point, @situx :

Everything seems fine except for this part Geographic information/Geomatics.
The tag should be a hyperlink pointing to an internet address, but no internet address is given. @Iza82

this anchor is automatically converted by ReSpec into a link to the corresponding definition

https://raw.githack.com/w3c/sdw/iza-review/bp/index.html#dfn-geographic-information

@Iza82
Copy link
Contributor Author

Iza82 commented Oct 7, 2022

@situx the link is an internal link to geographic information

@situx
Copy link
Collaborator

situx commented Oct 7, 2022

Okay, thanks all for the clarification. Then I would approve it and merge it

@situx situx self-requested a review October 7, 2022 14:35
@situx situx merged commit a797c9a into gh-pages Oct 20, 2022
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

Successfully merging this pull request may close these issues.

None yet

3 participants