Skip to content

Alexg local new - #73

Merged
Pixcell merged 5 commits into
stagingfrom
alexgLocalNew
Jul 5, 2022
Merged

Alexg local new#73
Pixcell merged 5 commits into
stagingfrom
alexgLocalNew

Conversation

@ag-edge

@ag-edge ag-edge commented Jun 22, 2022

Copy link
Copy Markdown
Contributor

Summary

Added links to words in terminology to either point to terminology.md or to their respective pages (Darcy cloud for example).

Also changed spelling/punctuation/grammar in various places where noticed.

Basic example

https://i.imgur.com/oWGV31H.png

Motivation

To make sure people have their definitions readily available for specific content.

Checks

  • Verify that your changes build locally. Passes:
    • npm run test
    • npm run build:preview followed by npm run lint
  • Create a Pull Request (follow the GitHub flow) and Conventional Commits Specification)
  • Supports all screen sizes (if relevant)
  • Supports both light and dark mode (if relevant)

Adding links to terminology and links to pages when words in terminology are mentioned in content. Limited to the first time said word(s) are mentioned.

Also fixed various grammar issues.
@netlify

netlify Bot commented Jun 22, 2022

Copy link
Copy Markdown

Deploy Preview for darcydocs ready!

Name Link
🔨 Latest commit 66f0067
🔍 Latest deploy log https://app.netlify.com/sites/darcydocs/deploys/62c4a25742fab6000989a2bc
😎 Deploy Preview https://deploy-preview-73--darcydocs.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.

…nks Multiple consecutive blank lines [Expected: 1; Actual: 2]

Resolved issue here content/en/docs/cloud/adding-nodes/_index.md:28 MD012/no-multiple-blanks Multiple consecutive blank lines [Expected: 1; Actual: 2]
@Pixcell
Pixcell merged commit ea9f372 into staging Jul 5, 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.

3 participants