Skip to content

Conversation

@marcoscaceres
Copy link
Contributor

@marcoscaceres marcoscaceres commented Feb 16, 2022

Complains if latestVersion doesn't exist, while it also prevents CG drafts from claiming to have a latestVersion.

Copy link
Member

@sidvishnoi sidvishnoi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will check in detail from computer.

@sidvishnoi sidvishnoi self-requested a review February 16, 2022 07:23
Comment on lines +37 to +43
${conf.latestVersion !== null
? html`<dt>${l10n.latest_published_version}</dt>
<dd>
${conf.latestVersion
${conf.latestVersion !== ""
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When is it null, and when is it "" in particular?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When it's null, it shouldn't be included... but when it's ""... I need to check :(

@netlify
Copy link

netlify bot commented Aug 6, 2025

Deploy Preview for respec-pr ready!

Name Link
🔨 Latest commit 8e701bd
🔍 Latest deploy log https://app.netlify.com/projects/respec-pr/deploys/68f0500191d6c90008bbd5ef
😎 Deploy Preview https://deploy-preview-4037--respec-pr.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 project configuration.

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.

3 participants