Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add return types QueryResult<T> to database helper functions #1419

Closed
marcocastignoli opened this issue Jun 4, 2024 · 0 comments · Fixed by #1451
Closed

Add return types QueryResult<T> to database helper functions #1419

marcocastignoli opened this issue Jun 4, 2024 · 0 comments · Fixed by #1451
Assignees

Comments

@marcocastignoli
Copy link
Member

In services/server/src/server/services/utils/database-util.ts every function returns a pg query QueryResult, we can improve the return type including the object that the pg Result is returning. E.g. QueryResult<RowType>

@marcocastignoli marcocastignoli moved this from Triage to Sprint - Up Next in Sourcify Public Jun 6, 2024
@marcocastignoli marcocastignoli self-assigned this Jun 6, 2024
@marcocastignoli marcocastignoli moved this from Sprint - Up Next to Sprint - In Progress in Sourcify Public Jun 6, 2024
@kuzdogan kuzdogan moved this from Sprint - In Progress to Sprint - Up Next in Sourcify Public Jun 10, 2024
@marcocastignoli marcocastignoli moved this from Sprint - Up Next to Sprint - In Progress in Sourcify Public Jun 18, 2024
@marcocastignoli marcocastignoli moved this from Sprint - In Progress to Backlog in Sourcify Public Jun 18, 2024
@marcocastignoli marcocastignoli moved this from Backlog to Sprint - Up Next in Sourcify Public Jun 18, 2024
@marcocastignoli marcocastignoli linked a pull request Jun 25, 2024 that will close this issue
@marcocastignoli marcocastignoli moved this from Sprint - Up Next to Sprint - In Progress in Sourcify Public Jun 25, 2024
@github-project-automation github-project-automation bot moved this from Sprint - In Progress to Sprint - Done in Sourcify Public Jun 26, 2024
@kuzdogan kuzdogan moved this from Sprint - Done to COMPLETED in Sourcify Public Jul 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

1 participant