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

[DOCS-269] External Pentest Methodology #270

Draft
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

mjang-cobalt
Copy link
Contributor

@mjang-cobalt mjang-cobalt commented Jan 6, 2023

Changelog

  • Updated internal-network.md and external-network.md files.
  • Included additional content, using internal sources approved for external use
  • Laid out common content in shortcodes, by subsection
  • Updated content (primarily in shortcodes) for readability
  • Updated other methodologies pages with common short codes

Updated

| Page | Deploy Preview | Comment |
| Internal Network Methodology | Internal | ----- |
| External Network Methodology | External | Comment |

Preview This Change

To see how this change looks in production, scroll down to Deploy Preview. Select the link that looks like https://deploy-preview-<num>--cobalt-docs.netlify.app/

Variables

Help us support a “Write once, publish everywhere” single source of truth. If you see a line that looks like:

{{% asset-categories %}}

You’ve found a shortcode that we include in multiple documents.

You’ll find the content of the shortcode in the following directory:

https://github.com/cobalthq/cobalt-product-public-docs/tree/main/layouts/shortcodes

That shortcode has the same base name as what you see in the PR, such as asset-categories.html.

Checklist for PR Author

[ ] Did you check for broken links and alt text?

Be sure to check for broken links and Alt Text issues. We have a partially automated process,
as described in this section of our repository README:
Test Links and Alt Attributes.

Link Check

Essentially OK, though links for external authors to LinkedIn return 999
Links to Zendesk return 403s (based on protection from auth)
htmltest "passed"

@mjang-cobalt mjang-cobalt added do-not-merge draft Not ready for review labels Jan 6, 2023
@mjang-cobalt mjang-cobalt self-assigned this Jan 6, 2023
@netlify
Copy link

netlify bot commented Jan 6, 2023

Deploy Preview for cobalt-docs ready!

Name Link
🔨 Latest commit ae68423
🔍 Latest deploy log https://app.netlify.com/sites/cobalt-docs/deploys/63b89b46f8b72b00081854e4
😎 Deploy Preview https://deploy-preview-270--cobalt-docs.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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do-not-merge draft Not ready for review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant