Skip to content

change: [M3-9735] - Add encryption icon to Image edit drawer #11993

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

Conversation

bill-akamai
Copy link
Contributor

@bill-akamai bill-akamai commented Apr 8, 2025

Description 📝

This PR adds the "Encrypted" label to the EditImageDrawer component.

Changes 🔄

  • Adds "Encrypted" label to Edit Image Drawer
  • Includes Lock icon next to text

Preview 📷

Before After

Verification steps

  • Navigate to /images and create or find an existing custom image.
  • Click the ellipsis menu next to a custom image and click Edit.
  • Confirm addition of "Encrypted" text and icon in the Edit Image drawer.
Author Checklists

As an Author, to speed up the review process, I considered 🤔

👀 Doing a self review
❔ Our contribution guidelines
🤏 Splitting feature into small PRs
➕ Adding a changeset
🧪 Providing/improving test coverage
🔐 Removing all sensitive information from the code and PR description
🚩 Using a feature flag to protect the release
👣 Providing comprehensive reproduction steps
📑 Providing or updating our documentation
🕛 Scheduling a pair reviewing session
📱 Providing mobile support
♿ Providing accessibility support


  • I have read and considered all applicable items listed above.

As an Author, before moving this PR from Draft to Open, I confirmed ✅

  • All unit tests are passing
  • TypeScript compilation succeeded without errors
  • Code passes all linting rules

@bill-akamai bill-akamai changed the title change: [M3-935] - Add encryption icon to Image edit drawer change: [M3-9735] - Add encryption icon to Image edit drawer Apr 8, 2025
@bill-akamai bill-akamai marked this pull request as ready for review April 8, 2025 18:29
@bill-akamai bill-akamai requested a review from a team as a code owner April 8, 2025 18:29
@bill-akamai bill-akamai requested review from bnussman-akamai, abailly-akamai and cpathipa and removed request for a team April 8, 2025 18:29
@linode-gh-bot
Copy link
Collaborator

Cloud Manager UI test results

🔺 3 failing tests on test run #3 ↗︎

❌ Failing✅ Passing↪️ Skipped🕐 Duration
3 Failing535 Passing6 Skipped112m 55s

Details

Failing Tests
SpecTest
object-storage-objects-multicluster.spec.tsCloud Manager Cypress Tests→Object Storage Multicluster objects » Object Storage Multicluster objects
object-storage.e2e.spec.tsCloud Manager Cypress Tests→object storage end-to-end tests » object storage end-to-end tests
access-key.e2e.spec.tsCloud Manager Cypress Tests→object storage access key end-to-end tests » object storage access key end-to-end tests

Troubleshooting

Use this command to re-run the failing tests:

pnpm cy:run -s "cypress/e2e/core/objectStorageMulticluster/object-storage-objects-multicluster.spec.ts,cypress/e2e/core/objectStorage/object-storage.e2e.spec.ts,cypress/e2e/core/objectStorage/access-key.e2e.spec.ts"

<Stack direction="row" spacing={1} alignItems="center">
<Lock />
<Typography
sx={(theme) => ({ color: theme.textColors.textAccessTable })}
Copy link
Contributor

Choose a reason for hiding this comment

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

Have you confirmed this is the right token to use? Not that I have a better suggestion 😄 , but the token name seems unintuitive considering the purpose.

Copy link
Contributor Author

@bill-akamai bill-akamai Apr 8, 2025

Choose a reason for hiding this comment

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

This appeared to be the closest we had. I don't believe we have a specific token for this grayish text label note (or whatever you'd call it 🤷).

@bill-akamai bill-akamai merged commit 9385fda into linode:develop Apr 8, 2025
27 of 28 checks passed
@bill-akamai bill-akamai self-assigned this Apr 14, 2025
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.

4 participants