Skip to content

Conversation

udayk-port
Copy link
Contributor

Description

Including images from port app for security solutions

Updated docs pages

  • /solutions/security/overview
  • /solutions/security/prioritise-vulnerabilities
  • /solutions/security/security-metrics-visualization
  • /solutions/security/security-champions-initiatives
  • /solutions/security/security-actions-automations

Copy link

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-2862.d2ngvl90zqbob8.amplifyapp.com

Copy link
Collaborator

@kodjomiles kodjomiles left a comment

Choose a reason for hiding this comment

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

Looks great Overall
Let's use the "img" tags for the images and control the width and border line. in your case the images are dark so it looks fine without the border lines but we can reduce the sizes so they don't look too big.
Not at 100% all the time. some of the images can be resized to 80% to look better in context

Comment on lines 163 to 165
- Trigger security scans based on code changes and deployment patterns
- Integrate security testing into CI/CD pipelines with intelligent fail/pass decisions
- Generate security test cases based on application architecture and attack patterns
Copy link
Collaborator

Choose a reason for hiding this comment

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

Periods at the end of every list item

Comment on lines 8 to 14
:::tip Transform metrics into actionable insights
**Port transforms security metrics from vanity numbers into actionable insights, helping CISOs, security teams, and developers align on what truly matters.**
:::

:::tip Unify security strategy and execution
**From prioritizing vulnerabilities to visualizing progress, Port unifies security strategy and execution in one powerful platform.**
:::
Copy link
Collaborator

Choose a reason for hiding this comment

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

We don't advise showing two different admonitions like this in a row
Can we find another way to communicate this ?

Comment on lines 2 to 5
description:
globs:
alwaysApply: false
globs: **/*.md
alwaysApply: true
---
Copy link
Collaborator

Choose a reason for hiding this comment

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

you modified the rules 😃
please revert this

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Reverting

Comment on lines 74 to 75
- Regulatory scope (SOC 2, PCI, HIPAA, GDPR),
- Data sensitivity (PII, payment data, internal-only),
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
- Regulatory scope (SOC 2, PCI, HIPAA, GDPR).
- Data sensitivity (PII, payment data, internal-only).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ah! I kept staring at this sentences for a very long time until I noticed my punctuation.

Copy link
Collaborator

@kodjomiles kodjomiles left a comment

Choose a reason for hiding this comment

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

LGTM

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