Skip to content

develop to main#19

Merged
JosueBrenes merged 5 commits intomainfrom
develop
Dec 18, 2025
Merged

develop to main#19
JosueBrenes merged 5 commits intomainfrom
develop

Conversation

@JosueBrenes
Copy link
Contributor

This pull request introduces a new API Key request feature to the dashboard, refactors credential verification to use the new API, and improves type definitions and UI interactions. It also updates dependencies and cleans up provider usage for better maintainability. The most important changes are grouped below:

API Key Feature Additions:

  • Added a new dashboard page (src/app/dashboard/api-keys/page.tsx) and UI component (ApiKeys) for requesting and displaying public API keys, including clipboard copy functionality. [1] [2]
  • Implemented usePublicApiKey hook for requesting API keys from the backend, handling state, errors, and storing the key for use across the app.
  • Defined new types for API key records and responses (src/@types/api-keys.ts).

Credential Verification Refactor:

  • Refactored credential verification to use the new API endpoint with API key authentication, removing legacy SDK and on-chain verification code. [1] [2] [3] [4] [5]
  • Added raw and vaultRecord fields to the Credential type and included them in the credential adapter for improved credential content handling. [1] [2]

UI and UX Improvements:

  • Added onView handler to CredentialCardProps and included an eye icon in the credential card UI for viewing credential content. [1] [2] [3] [4]

Configuration and Dependency Updates:

  • Updated @acta-team/acta-sdk dependency to version ^1.0.5 in package.json.
  • Updated Next.js dev/build scripts to use webpack and silenced noisy warnings from optional native dependencies in next.config.ts. [1] [2]

Provider and Type Cleanup:

  • Removed unused ActaProvider from the app layout and cleaned up environment and on-chain types for clarity and consistency. [1] [2] [3] [4] [5]## Description

Type of Change

  • 🐛 Bug fix (non-breaking change which fixes an issue)
  • ✨ New feature (non-breaking change which adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • 📚 Documentation update
  • 🎨 Style/formatting changes (no code change)
  • ♻️ Code refactoring (no functional changes)
  • ⚡ Performance improvement
  • ✅ Tests (adding missing tests or correcting existing tests)
  • 🔧 Build/config changes
  • 🔒 Security fix

Related Issues

Closes #
Fixes #
Relates to #

Changes Made

Screenshots / Videos

Before After

Testing

Test Cases

  • Test case 1:
  • Test case 2:
  • Test case 3:

Manual Testing Steps

Checklist

  • My code follows the project's style guidelines
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published
  • I have checked for breaking changes and documented them if applicable
  • I have updated the CHANGELOG.md (if applicable)

Dependencies

Added

Removed

Updated

Additional Notes

Reviewers

/cc @

- Added a new API Keys page for users to request and manage their API keys.
- Introduced hooks for handling API key requests and responses.
- Updated sidebar navigation to include a link to the new API Keys page.
- Bumped the version of @acta-team/acta-sdk to 1.0.4 and modified the build scripts to use webpack.
- Enhanced webpack configuration to ignore warnings from optional native dependencies in stellar-sdk.
- Removed unnecessary blank lines in api-keys type definitions and API keys page component for cleaner code.
- Bumped @acta-team/acta-sdk to version 1.0.5 in package.json, package-lock.json, and yarn.lock for improved functionality.
- Removed the deprecated acta.provider.tsx file and refactored related components to utilize direct API calls for credential issuance and verification.
- Enhanced the API key management feature by adding input fields for API key entry in the issue credential form and ensuring proper handling in vault operations.
- Updated the sidebar to include a link to the API Keys page and improved error handling in various hooks related to credential management.
- Removed unnecessary blank lines in api-keys type definitions and API keys page component for cleaner code.
- Added a temporary notice in the NetworkSwitchModal to inform users that mainnet is currently disabled while API and contract updates are in progress.
- Translated the temporary notice in the NetworkSwitchModal to English, clarifying that Mainnet is temporarily unavailable due to ongoing API and contract updates.
@vercel
Copy link
Contributor

vercel bot commented Dec 18, 2025

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

Project Deployment Review Updated (UTC)
dapp Ready Ready Preview, Comment Dec 18, 2025 9:52pm

@coderabbitai
Copy link

coderabbitai bot commented Dec 18, 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.

@JosueBrenes JosueBrenes merged commit 50ba9e0 into main Dec 18, 2025
1 of 2 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.

1 participant