Skip to content

develop to main#24

Merged
JosueBrenes merged 2 commits intomainfrom
develop
Dec 19, 2025
Merged

develop to main#24
JosueBrenes merged 2 commits intomainfrom
develop

Conversation

@JosueBrenes
Copy link
Contributor

This pull request enhances the API key creation flow by improving security messaging, enforcing wallet requirements, and clarifying the user interface. The changes ensure that users must connect their wallet before creating an API key, provide clear instructions and warnings about API key management, and improve the overall UX for API key creation and retrieval.

User experience and security improvements:

  • Added a prominent security warning box in the ApiKeys UI, outlining key rules: only one API key per wallet, the key is shown only once, users must save it immediately, and the key is tied to wallet access control.
  • Added contextual success, error, and informational messages to guide users through the API key creation process, including a success alert when the key is created and a prompt to connect a wallet if not already connected.

Enforcing wallet requirements:

  • Updated the usePublicApiKey hook to require a wallet address before creating an API key, throwing an error if not present and always including wallet_address in the API request payload.
  • The API key creation button is now disabled if the wallet is not connected or if an API key has already been created, and the button label updates dynamically based on state.

UI/UX refinements:

  • Improved copy and instructions throughout the API key UI, including clarifying that the API key will only be shown once and updating section titles for clarity. [1] [2]
  • Added the wallet context to the ApiKeys component to manage wallet connection state and control UI behavior accordingly. [1] [2]

JosueBrenes and others added 2 commits December 18, 2025 21:13
…urity warnings

- Added wallet address validation to ensure users connect their wallet before creating an API key.
- Introduced a security warning section detailing important information about API key usage and storage.
- Updated button states and messages based on wallet connection status and API key creation status for better user experience.
@JosueBrenes JosueBrenes merged commit 430c62d into main Dec 19, 2025
2 of 3 checks passed
@coderabbitai
Copy link

coderabbitai bot commented Dec 19, 2025

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch develop

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@vercel
Copy link
Contributor

vercel bot commented Dec 19, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
dapp Building Building Preview, Comment Dec 19, 2025 5:43am

JosueBrenes added a commit that referenced this pull request Dec 19, 2025
Merge pull request #24 from ACTA-Team/develop
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.

1 participant