w3c / wot-binding-templates

Web of Things (WoT) Binding Templates
http://w3c.github.io/wot-binding-templates/
Other
23 stars 25 forks source link

Publication Prep #290

Closed egekorkan closed 1 year ago

egekorkan commented 1 year ago

I have done the following:

Validation results (No need to manually update):


Preview | Diff

netlify[bot] commented 1 year ago

Deploy Preview for wot-binding-templates ready!

Name Link
Latest commit 3563204f89f95bfa76f5e03f1b114fe9965a18ea
Latest deploy log https://app.netlify.com/sites/wot-binding-templates/deploys/646fd62b122a020008262c48
Deploy Preview https://deploy-preview-290--wot-binding-templates.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

egekorkan commented 1 year ago

The Netlify version can be used for the pubrules check: https://deploy-preview-290--wot-binding-templates.netlify.app/publication/note3/overview

egekorkan commented 1 year ago

Pubrules error: [EXCEPTION] The document "Web of Things (WoT) Binding Templates" seems to be an Editor's Draft, which is not supported.

It is not clear from previous instructions on what needs to be changed to move from ED to Note

egekorkan commented 1 year ago

Call of 24.05:

egekorkan commented 1 year ago

All the checks except some CSS that is generated by respec are passing. @danielpeintner did you have the same issue in Scripting API? You can find them at the CSS validator or in the HTML validator of W3C.

danielpeintner commented 1 year ago

@danielpeintner did you have the same issue in Scripting API?

We did run into some other ReSpec issues, see https://github.com/w3c/wot-scripting-api/issues/480

egekorkan commented 1 year ago

OK so I checked where the error comes from and it is not due to respec, also not the errors like in @danielpeintner case above. The static HTML generated from respec is correct but HTML tidy adds errors.

CSS validation of static.html

egekorkan commented 1 year ago

The addition of CDATA seems to be a bug that others counter (see here). I will go around this issue for now.