Skip to content

improvement: clarify authorized emails must have an infisical account for secret sharing#5343

Merged
scott-ray-wilson merged 1 commit intomainfrom
secret-share-authorized-email-clarification
Feb 3, 2026
Merged

improvement: clarify authorized emails must have an infisical account for secret sharing#5343
scott-ray-wilson merged 1 commit intomainfrom
secret-share-authorized-email-clarification

Conversation

@scott-ray-wilson
Copy link
Contributor

Context

This PR adds helper text and tooltip text to clarify that authorized email recipients for secret sharing must have an Infisical account

Screenshots

CleanShot 2026-02-02 at 12 04 18@2x

Steps to verify the change

Perceive the text with your eyes

Type

  • Fix
  • Feature
  • Improvement
  • Breaking
  • Docs
  • Chore

Checklist

  • Title follows the conventional commit format: type(scope): short description (scope is optional, e.g., fix: prevent crash on sync or fix(api): handle null response).
  • Tested locally
  • Updated docs (if needed)
  • Read the contributing guide

@maidul98
Copy link
Collaborator

maidul98 commented Feb 2, 2026

Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Feb 2, 2026

Greptile Overview

Greptile Summary

This PR improves user experience by adding clarifying text to the "Authorized Emails" field in the secret sharing form. The changes help users understand that recipients must have an Infisical account before they can access shared secrets.

Changes:

  • Added inline helper text below the "Authorized Emails" input field
  • Enhanced the tooltip with multi-paragraph explanation of the requirement

The changes are purely cosmetic UI improvements with no functional or security impact. The implementation follows React best practices and is safe to merge.

Confidence Score: 5/5

  • This PR is safe to merge with no risk
  • The change only adds clarifying UI text to help users understand requirements. No logic, security, or functional changes were made.
  • No files require special attention

Important Files Changed

Filename Overview
frontend/src/pages/public/ShareSecretPage/components/ShareSecretForm.tsx Added helper text and expanded tooltip to clarify that recipients need an Infisical account for secret sharing

@scott-ray-wilson scott-ray-wilson merged commit ffdd35a into main Feb 3, 2026
8 of 11 checks passed
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