Skip to content

fix(api): remove duplicate CredentialNFT import and unused CredentialMetadata (#520) - #526

Open
lmduy2612 wants to merge 1 commit into
ChainLearnOfficial:mainfrom
lmduy2612:fix/credentials-duplicate-imports-520
Open

lmduy2612 wants to merge 1 commit into
ChainLearnOfficial:mainfrom
lmduy2612:fix/credentials-duplicate-imports-520

Conversation

@lmduy2612

Copy link
Copy Markdown

Summary

Closes #520.

Removes duplicate CredentialNFT type import and unused CredentialMetadata type import from src/lib/api/credentials.ts, while organizing top-level imports cleanly.

Changes Made

  • Consolidated import statements at the top of src/lib/api/credentials.ts.
  • Removed line 24 duplicate CredentialNFT and unused CredentialMetadata.
  • Verified that all unit tests in src/lib/api/credentials.test.ts pass 100% and ESLint reports zero errors.

Acceptance Criteria

  • Remove line 3 (duplicate CredentialNFT import) / consolidate cleanly
  • Remove CredentialMetadata from the import
  • File compiles without errors
  • No unused import warnings from ESLint/TypeScript

@netlify

netlify Bot commented Oct 2, 2026 •

Copy link
Copy Markdown

❌ Deploy Preview for chainlearn failed.

Name Link
🔨 Latest commit 295500c
🔍 Latest deploy log https://app.netlify.com/projects/chainlearn/deploys/6abf70b267312100083eec38

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.

Duplicate imports in credentials.ts — CredentialNFT imported twice, CredentialMetadata unused

1 participant