feat(platform): Improved UI of secret listing

[rajdip-b]

User description

Description

Fixes #622

---

PR Type

Enhancement

---

Description

• Updated secret listing to display "Hidden" for non-decrypted values.

• Enhanced environment display format to include name and slug.

• Changed "Secret" tab header to "Value".

• Displayed user name instead of ID in "last edited by".

---

Changes walkthrough 📝

Relevant files

Enhancement

page.tsx Enhance secret listing logic and API integration                  apps/platform/src/app/(main)/project/[project]/@secret/page.tsx Added isDecrypted logic to determine secret visibility. Updated API call to include decryptValue parameter. Adjusted dependencies in useEffect for reactivity. Modified secret mapping to pass isDecrypted and updated props. +13/-5    index.tsx Update secret card display and formatting                                apps/platform/src/components/dashboard/secret/secretCard/index.tsx Introduced isDecrypted prop to control secret visibility. Updated environment display to include name and slug. Changed "Secret" tab header to "Value". Displayed user name instead of ID in "last edited by". +17/-5

---

Need help?

Type /help how to ... in the comments thread for any questions about Qodo Merge usage.

Check out the documentation for more information.

[codiumai-pr-agent-free]

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

🎫 Ticket compliance analysis ✅ 622 - PR Code Verified Compliant requirements: Show "Hidden" for non-decrypted secret values instead of buffer Display environment in format: "Environment Name (environment slug)" Rename "Secret" tab to "Value" Show user name instead of ID in "last edited by" Requires further human verification: Verify that "Hidden" text appears correctly for non-decrypted values in UI Validate environment name and slug display format in UI

⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪

🧪 No relevant tests

🔒 No security concerns identified

⚡ Recommended focus areas for review Error Handling The error case in the API call is not properly handled - error message is not displayed to user when API call fails const { success, error, data } = await ControllerInstance.getInstance().secretController.getAllSecretsOfProject( { projectSlug: selectedProject.slug, decryptValue: isDecrypted }, {} )

[codiumai-pr-agent-free]

PR Code Suggestions ✨

Explore these optional code suggestions:

Category	Suggestion	Score
Possible issue	Fix async loading state handling Move setIsLoading(false) inside the getAllSecretsByProjectSlug function after the API call completes to prevent premature loading state changes. apps/platform/src/app/(main)/project/[project]/@secret/page.tsx [61-63] +async function getAllSecretsByProjectSlug() { + if (!selectedProject?.slug) { + setIsLoading(false) + return + } + const { success, data } = await ControllerInstance.getInstance().secretController.getAllSecretsOfProject( + { projectSlug: selectedProject.slug, decryptValue: isDecrypted }, + {} + ) + if (success && data) { + setSecrets(data) + } + setIsLoading(false) +} getAllSecretsByProjectSlug() -setIsLoading(false) - Apply this suggestion Suggestion importance[1-10]: 8 Why: Moving setIsLoading(false) inside the async function is important for preventing race conditions and ensuring the loading state accurately reflects the API call completion status.	8
	Add error state handling Add error handling for failed API responses to properly inform users and handle error states. apps/platform/src/app/(main)/project/[project]/@secret/page.tsx [47-53] const { success, error, data } = await ControllerInstance.getInstance().secretController.getAllSecretsOfProject( { projectSlug: selectedProject.slug, decryptValue: isDecrypted }, {} ) if (success && data) { setSecrets(data) +} else if (error) { + setSecrets([]) + toast.error(`Failed to load secrets: ${error.message}`) } Apply this suggestion Suggestion importance[1-10]: 7 Why: Adding proper error handling with user feedback is important for reliability and UX, though the error case is already partially handled by clearing the secrets array.	7

[kriptonian1]

LGTM 🚀

[rajdip-b]

🎉 This PR is included in version 2.10.0-stage.3 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

[rajdip-b]

🎉 This PR is included in version 2.10.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀