Skip to content

Comments

fix(v2): accept empty/null custom_edit_url docs frontmatter for retrocompat#4687

Merged
slorber merged 1 commit intomasterfrom
slorber/accept-empty-custom_edit_url
Apr 27, 2021
Merged

fix(v2): accept empty/null custom_edit_url docs frontmatter for retrocompat#4687
slorber merged 1 commit intomasterfrom
slorber/accept-empty-custom_edit_url

Conversation

@slorber
Copy link
Collaborator

@slorber slorber commented Apr 27, 2021

Motivation

We used to accept "" as valid custom_edit_url value before adding the frontmatter validation system.

Let's keep it this way to avoid annoying existing users (see demisto/content-docs#616 (comment))

Have you read the Contributing Guidelines on pull requests?

yes

Test Plan

test

@slorber slorber added the pr: bug fix This PR fixes a bug in a past release. label Apr 27, 2021
@slorber slorber requested a review from lex111 as a code owner April 27, 2021 10:04
@facebook-github-bot facebook-github-bot added the CLA Signed Signed Facebook CLA label Apr 27, 2021
@netlify
Copy link

netlify bot commented Apr 27, 2021

@netlify
Copy link

netlify bot commented Apr 27, 2021

@github-actions
Copy link

⚡️ Lighthouse report for the changes in this PR:

Category Score
🟠 Performance 67
🟢 Accessibility 96
🟢 Best practices 100
🟢 SEO 100
🟢 PWA 95

Lighthouse ran on https://deploy-preview-4687--docusaurus-2.netlify.app/

@github-actions
Copy link

Size Change: 0 B

Total Size: 631 kB

ℹ️ View Unchanged
Filename Size Change
website/build/assets/css/styles.********.css 88 kB 0 B
website/build/assets/js/main.********.js 453 kB 0 B
website/build/blog/2017/12/14/introducing-docusaurus/index.html 62.3 kB 0 B
website/build/docs/introduction/index.html 235 B 0 B
website/build/index.html 27.1 kB 0 B

compressed-size-action

@slorber slorber merged commit 8efc0cd into master Apr 27, 2021
@slorber slorber deleted the slorber/accept-empty-custom_edit_url branch August 17, 2021 17:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed Signed Facebook CLA pr: bug fix This PR fixes a bug in a past release.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants