Skip to content

Feature/governance labels integration#10100

Draft
mgallien wants to merge 5 commits into
masterfrom
feature/governanceLabelsIntegration
Draft

Feature/governance labels integration#10100
mgallien wants to merge 5 commits into
masterfrom
feature/governanceLabelsIntegration

Conversation

@mgallien
Copy link
Copy Markdown
Collaborator

@mgallien mgallien commented Jun 2, 2026

Thanks for opening a pull request on the Nextcloud desktop client.

Before sending a pull request that fixes a security issue please report it via our HackerOne page (https://hackerone.com/nextcloud) following our security policy (https://nextcloud.com/security/).
This allows us to coordinate the fix and release without potentially exposing all Nextcloud client users in the meantime.

Instead of a Contributor License Agreement (CLA) we use a Developer Certificate of Origin (DCO).
https://en.wikipedia.org/wiki/Developer_Certificate_of_Origin
-->

Resolves

#

Close https://github.com/nextcloud-gmbh/server/issues/953
Close https://github.com/nextcloud-gmbh/server/issues/957

Summary

TODO

  • ...

Checklist

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

@mgallien mgallien added feature: ☁️ GUI System tray icon and menu. 2. developing labels Jun 2, 2026
@mgallien mgallien added this to the 34.0.0 milestone Jun 2, 2026
Copy link
Copy Markdown
Contributor

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

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

⚠️ Clang-Tidy found issue(s) with the introduced code (1/1)


void setCustomEntityType(const QString &newCustomEntityType);

QString entityId() const;
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

⚠️ modernize-use-nodiscard ⚠️
function entityId should be marked [[nodiscard]]

Suggested change
QString entityId() const;
[[nodiscard]] QString entityId() const;

mgallien added 4 commits June 5, 2026 09:01
will enable to share type definitions in all real jobs

Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
also improve the available properties for the job parameters

Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
finish implementing most of the code needed to send requests to the
governance API

will need further work to see how to read the replies and make something
usefull

Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
@mgallien mgallien force-pushed the feature/governanceLabelsIntegration branch from 08ca064 to 1582c09 Compare June 5, 2026 07:01
@github-actions github-actions Bot dismissed their stale review June 5, 2026 07:13

No Clang-Tidy warnings found so I assume my comments were addressed

@mgallien mgallien force-pushed the feature/governanceLabelsIntegration branch from 1582c09 to 9af8192 Compare June 5, 2026 07:21
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Jun 5, 2026

Quality Gate Failed Quality Gate failed

Failed conditions
0.0% Coverage on New Code (required ≥ 80%)
D Security Rating on New Code (required ≥ A)
128 New Code Smells (required ≤ 0)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
@mgallien mgallien force-pushed the feature/governanceLabelsIntegration branch from 9af8192 to b830eeb Compare June 5, 2026 09:29
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 5, 2026

Artifact containing the AppImage: nextcloud-appimage-pr-10100.zip

Digest: sha256:b16189b2063c11999e9999737b0df3e5c8cec5a32a829ea143c706af6d26bc78

To test this change/fix you can download the above artifact file, unzip it, and run it.

Please make sure to quit your existing Nextcloud app and backup your data.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2. developing feature: ☁️ GUI System tray icon and menu.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant