Skip to content

feat: split api tokens into separate table#540

Open
mfontanini wants to merge 1 commit intomainfrom
feat/api-tokens
Open

feat: split api tokens into separate table#540
mfontanini wants to merge 1 commit intomainfrom
feat/api-tokens

Conversation

@mfontanini
Copy link
Copy Markdown
Contributor

This moves nilcc-api's api tokens into a separate table so we can have more than one token per account. For now the original api_token field is still there, I'll remove that in the next change.

@github-actions
Copy link
Copy Markdown

Coverage Report for nilcc-api

Status Category Percentage Covered / Total
🔵 Lines 86.73% 3883 / 4477
🔵 Statements 86.73% 3883 / 4477
🔵 Functions 82.77% 197 / 238
🔵 Branches 81.66% 343 / 420
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
nilcc-api/migrations/1771977600000-ApiTokensTable.ts 61.53% 100% 66.66% 61.53% 33-42
nilcc-api/src/data-source.ts 84.76% 84.21% 72.72% 84.76% 99-100, 112-113, 156-157, 164-174, 177-182
nilcc-api/src/account/account.controller.ts 95.5% 92.3% 100% 95.5% 143-144, 206-212
nilcc-api/src/account/account.entity.ts 100% 100% 100% 100%
nilcc-api/src/account/account.service.ts 75.77% 69.56% 88.88% 75.77% 59-60, 71-72, 92-93, 103-114, 124-128, 133-135, 145-147, 153-162
nilcc-api/src/api-token/api-token.entity.ts 100% 100% 100% 100%
Generated in workflow #176 for commit bc2303d by the Vitest Coverage Report Action

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