Merge pull request #60360 from nextcloud/fix/no-rate-shipped
Some checks are pending
CodeQL Advanced / Analyze (actions) (push) Waiting to run
CodeQL Advanced / Analyze (javascript-typescript) (push) Waiting to run
Integration sqlite / changes (push) Waiting to run
Integration sqlite / integration-sqlite (master, main, 8.4, main, --tags ~@large files_features) (push) Blocked by required conditions
Integration sqlite / integration-sqlite (master, main, 8.4, main, capabilities_features) (push) Blocked by required conditions
Integration sqlite / integration-sqlite (master, main, 8.4, main, collaboration_features) (push) Blocked by required conditions
Integration sqlite / integration-sqlite (master, main, 8.4, main, comments_features) (push) Blocked by required conditions
Integration sqlite / integration-sqlite (master, main, 8.4, main, dav_features) (push) Blocked by required conditions
Integration sqlite / integration-sqlite (master, main, 8.4, main, features) (push) Blocked by required conditions
Integration sqlite / integration-sqlite (master, main, 8.4, main, federation_features) (push) Blocked by required conditions
Integration sqlite / integration-sqlite (master, main, 8.4, main, file_conversions) (push) Blocked by required conditions
Integration sqlite / integration-sqlite (master, main, 8.4, main, files_reminders) (push) Blocked by required conditions
Integration sqlite / integration-sqlite (master, main, 8.4, main, filesdrop_features) (push) Blocked by required conditions
Integration sqlite / integration-sqlite (master, main, 8.4, main, guests_features) (push) Blocked by required conditions
Integration sqlite / integration-sqlite (master, main, 8.4, main, ldap_features) (push) Blocked by required conditions
Integration sqlite / integration-sqlite (master, main, 8.4, main, openldap_features) (push) Blocked by required conditions
Integration sqlite / integration-sqlite (master, main, 8.4, main, openldap_numerical_features) (push) Blocked by required conditions
Integration sqlite / integration-sqlite (master, main, 8.4, main, remoteapi_features) (push) Blocked by required conditions
Integration sqlite / integration-sqlite (master, main, 8.4, main, routing_features) (push) Blocked by required conditions
Integration sqlite / integration-sqlite (master, main, 8.4, main, setup_features) (push) Blocked by required conditions
Integration sqlite / integration-sqlite (master, main, 8.4, main, sharees_features) (push) Blocked by required conditions
Integration sqlite / integration-sqlite (master, main, 8.4, main, sharing_features) (push) Blocked by required conditions
Integration sqlite / integration-sqlite (master, main, 8.4, main, theming_features) (push) Blocked by required conditions
Integration sqlite / integration-sqlite (master, main, 8.4, main, videoverification_features) (push) Blocked by required conditions
Integration sqlite / integration-sqlite-summary (push) Blocked by required conditions
Psalm static code analysis / changes (push) Waiting to run
Psalm static code analysis / static-code-analysis (push) Blocked by required conditions
Psalm static code analysis / static-code-analysis-security (push) Blocked by required conditions
Psalm static code analysis / static-code-analysis-ocp (push) Blocked by required conditions
Psalm static code analysis / static-code-analysis-ncu (push) Blocked by required conditions
Psalm static code analysis / static-code-analysis-strict (push) Blocked by required conditions
Psalm static code analysis / static-code-analysis-summary (push) Blocked by required conditions

fix(appstore): do not offer rating apps not on appstore
This commit is contained in:
Ferdinand Thiessen 2026-05-15 20:55:11 +02:00 committed by GitHub
commit 1e833cea34
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
6 changed files with 4 additions and 6 deletions

View file

@ -478,7 +478,6 @@ class ApiController extends OCSController {
'active' => $this->appManager->isEnabledForUser($app['id']),
'needsDownload' => !$existsLocally,
'groups' => $groups,
'fromAppStore' => true,
'appstoreData' => $app,
];
}

View file

@ -17,7 +17,7 @@ export const actionsInteract: AppAction[] = [
inline: false,
label: () => t('appstore', 'Rate the app'),
enabled(app: IAppstoreApp | IAppstoreExApp) {
return !!app.fromAppStore
return !app.shipped
},
href(app: IAppstoreApp | IAppstoreExApp) {
return `https://apps.nextcloud.com/apps/${encodeURIComponent(app.id)}#comments`

View file

@ -69,7 +69,6 @@ interface IAppInfoData {
* It is not available for non-appstore apps.
*/
interface IAppstoreMetadata {
fromAppStore: true
/** List of appstore release information (e.g. changelog) */
releases: IAppstoreAppRelease[]
/** The overall rating of the app */

View file

@ -15,7 +15,7 @@ const { app } = defineProps<{
app: IAppstoreApp | IAppstoreExApp
}>()
const isShown = computed(() => app.ratingNumOverall && app.ratingNumOverall > 5)
const isShown = computed(() => !app.shipped && app.ratingNumOverall && app.ratingNumOverall > 5)
const score = computed(() => app.ratingOverall ?? 4)
const title = computed(() => {

File diff suppressed because one or more lines are too long

File diff suppressed because one or more lines are too long