Skip to content

feat(intake): port to v1betaapi#1364

Merged
Fyusel merged 5 commits intostackitcloud:mainfrom
stackit-intake:switch-intake-new-SDK
Apr 27, 2026
Merged

feat(intake): port to v1betaapi#1364
Fyusel merged 5 commits intostackitcloud:mainfrom
stackit-intake:switch-intake-new-SDK

Conversation

@yago-123
Copy link
Copy Markdown
Contributor

@yago-123 yago-123 commented Apr 10, 2026

Description

This PR migrates the current STACKIT Intake implementation to the new SDK hierarchy structure. In our case we must start using v1betaapi.

In order to perform this port we had to:

  1. Update all SDK import paths of Intake to v1betaapi
  2. Explicit use of DefaultAPI instead of using directly the client
  3. Shift from pointers to values for required fields
  4. Fix go-cmp to ensure unexported private fields do not affect the test output
  5. Adjust some linting changes

Checklist

  • Issue was linked above (not needed)
  • Code format was applied: make fmt
  • Examples were added / adjusted (see e.g. here) (not needed)
  • Docs are up-to-date: make generate-docs (will be checked by CI)
  • Unit tests got implemented or updated
  • Unit tests are passing: make test (will be checked by CI)
  • No linter issues: make lint (will be checked by CI)

@yago-123 yago-123 requested a review from a team as a code owner April 10, 2026 08:58
@yago-123 yago-123 force-pushed the switch-intake-new-SDK branch from 4471d4e to 38f30fb Compare April 10, 2026 09:05
@github-actions
Copy link
Copy Markdown

This PR was marked as stale after 7 days of inactivity and will be closed after another 7 days of further inactivity. If this PR should be kept open, just add a comment, remove the stale label or push new commits to it.

rubenhoenle
rubenhoenle previously approved these changes Apr 27, 2026
Fyusel
Fyusel previously approved these changes Apr 27, 2026
@Fyusel Fyusel dismissed stale reviews from rubenhoenle and themself via 55bd704 April 27, 2026 10:02
@Fyusel Fyusel force-pushed the switch-intake-new-SDK branch 2 times, most recently from 55bd704 to a36cde3 Compare April 27, 2026 10:05
Fyusel
Fyusel previously approved these changes Apr 27, 2026
@Fyusel Fyusel force-pushed the switch-intake-new-SDK branch from ff4e18f to a5dbc7b Compare April 27, 2026 10:18
@Fyusel Fyusel merged commit 1e8c409 into stackitcloud:main Apr 27, 2026
3 checks passed
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.

4 participants