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

change: API: rename Key to AppKey #844

Merged
merged 3 commits into from
Oct 31, 2024
Merged

change: API: rename Key to AppKey #844

merged 3 commits into from
Oct 31, 2024

Conversation

kjetilhau
Copy link
Member

@kjetilhau kjetilhau commented Oct 30, 2024

Description

Rename two API models with the following changes:

  • ApiPortalApp: Key is now AppKey

  • ApiPortalOnboardedApp: Key is now AppKey

  • 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

Renames Key to AppKey in ApiPortallApp and ApiPortalOnboardedApp models

Copy link

changeset-bot bot commented Oct 30, 2024

🦋 Changeset detected

Latest commit: 172e563

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

This PR includes changesets to release 1 package
Name Type
fusion-project-portal Minor

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 changed the title improvement: backend refactoring and improvements change: API: rename Key to AppKey Oct 30, 2024
@kjetilhau kjetilhau requested a review from Noggling October 30, 2024 13:13
@kjetilhau kjetilhau marked this pull request as ready for review October 30, 2024 13:13
@kjetilhau kjetilhau requested a review from a team as a code owner October 30, 2024 13:13
@kjetilhau kjetilhau merged commit d814023 into main Oct 31, 2024
3 checks passed
@kjetilhau kjetilhau deleted the api-refactoring branch October 31, 2024 09:22
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