Skip to content

Conversation

@TheBigLee
Copy link
Member

@TheBigLee TheBigLee commented Mar 24, 2025

The update function for the objectsuser requires an empty tag list instead of nil if we don't want any tags set.

This PR fixes this issue

Checklist

  • The PR has a meaningful title. It will be used to auto generate the
    changelog.
    The PR has a meaningful description that sums up the change. It will be
    linked in the changelog.
  • PR contains a single logical change (to build a better changelog).
  • Update the documentation.
  • Categorize the PR by adding one of the labels:
    bug, enhancement, documentation, change, breaking, dependency
    as they show up in the changelog.
  • Link this PR to related issues or PRs.

@TheBigLee TheBigLee added patch bug Something isn't working labels Mar 24, 2025
@TheBigLee TheBigLee requested review from a team, Kidswiss, ThisIsntTheWay, wejdross and zugao and removed request for a team and ThisIsntTheWay March 24, 2025 15:26
Copy link
Contributor

@Kidswiss Kidswiss left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess they recently changed that?

@zugao
Copy link
Contributor

zugao commented Mar 24, 2025

Maybe it's worth contacting cloudscale?

The update function for the objectsuser requires an empty tag list
instead of nil if we don't want any tags set.

This PR fixes this issue

Signed-off-by: Nicolas Bigler <nicolas.bigler@vshn.ch>
Signed-off-by: Nicolas Bigler <nicolas.bigler@vshn.ch>
@TheBigLee TheBigLee force-pushed the fix/usersobjects_update branch from 511261b to fe097f2 Compare March 24, 2025 15:51
@TheBigLee TheBigLee merged commit 2b7e184 into master Mar 24, 2025
7 checks passed
@TheBigLee TheBigLee deleted the fix/usersobjects_update branch March 24, 2025 15:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working patch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants