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

improvement: refactoring and various improvements #848

Merged
merged 11 commits into from
Nov 5, 2024
Merged

Conversation

kjetilhau
Copy link
Member

@kjetilhau kjetilhau commented Oct 31, 2024

Description

Continuation of improving and refactoring the Project Portal API and ClientBackend

  • Improve handling of nullable fields, made some fields non-nullable
  • Simplification and minor refactoring
  • Made cache duration configurable through AppSettings
  • Remove unused methods and properties
  • Remove unused cache configuration in ClientBackend
  • PR title and description are to the point and understandable
  • I have performed a self-review of my own code'

Please select version type the purposed change:

  • major
  • minor
  • patch
  • none

External Relations

  • database migration

Changeset

@kjetilhau kjetilhau self-assigned this Oct 31, 2024
Copy link

changeset-bot bot commented Oct 31, 2024

🦋 Changeset detected

Latest commit: 1a7b851

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 0 packages

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@kjetilhau kjetilhau requested a review from Noggling November 4, 2024 14:20
@kjetilhau kjetilhau marked this pull request as ready for review November 4, 2024 14:20
@kjetilhau kjetilhau requested a review from a team as a code owner November 4, 2024 14:20
@kjetilhau kjetilhau merged commit dad524f into main Nov 5, 2024
3 checks passed
@kjetilhau kjetilhau deleted the api-refactoring2 branch November 5, 2024 08:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants