Skip to content

fix(apps): persist editable pinned shortcuts - #393

Merged
veryCrunchy merged 10 commits into
mainfrom
fix/home-pinned-apps
Aug 16, 2026
Merged

fix(apps): persist editable pinned shortcuts#393
veryCrunchy merged 10 commits into
mainfrom
fix/home-pinned-apps

Conversation

@veryCrunchy

Copy link
Copy Markdown
Member

Summary

  • replace hard-coded pinned-app badges with explicit Pin and Unpin controls
  • persist a bounded ordered pin list per opaque account scope with safe fallback for invalid data
  • reuse pinned order for Home quick actions and desktop sidebar shortcuts
  • keep Talk and Spreed aliases canonical without duplicating shortcuts

Advances #183

Validation

Run on the dedicated ssh build host from a clean origin/main worktree:

  • :ui:desktopTest (2,471 tests; 1 pre-existing skip)
  • :ui:createDistributable
  • :androidApp:testDebugUnitTest
  • :androidApp:assembleDebug
  • bash tools/check-repository.sh
  • bash tools/test-prerelease-update-contract.sh

@obiente-cloud
obiente-cloud Bot temporarily deployed to Obiente Preview / PR #393 / NC Native August 16, 2026 13:53 Destroyed
@obiente-cloud

obiente-cloud Bot commented Aug 16, 2026

Copy link
Copy Markdown

Obiente preview

NC Native · 7ef14edc6219 · Removed

View preview status

View in Obiente

Obiente updates this comment as the preview changes.

@obiente-cloud
obiente-cloud Bot temporarily deployed to Obiente Preview / PR #393 / NC Native August 16, 2026 13:59 Destroyed

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: ff2b76e6cf

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread ui/src/commonMain/kotlin/dev/obiente/nextcloudnative/app/AppsWorkspace.kt Outdated
Comment thread ui/src/commonMain/kotlin/dev/obiente/nextcloudnative/app/NextcloudNativeApp.kt Outdated
Comment thread ui/src/commonMain/kotlin/dev/obiente/nextcloudnative/app/NextcloudNativeApp.kt Outdated
Comment thread ui/src/commonMain/kotlin/dev/obiente/nextcloudnative/app/AppWorkspacePins.kt Outdated
@veryCrunchy veryCrunchy added enhancement New feature or request priority:P1 Required for the pre-release product experience area:dynamic-apps Adaptive contracts, semantic UI, and app integrations area:ux Navigation, accessibility, responsive design, and interaction type:feature New product capability platform:android Android-specific implementation or validation platform:desktop Desktop-specific implementation or validation labels Aug 16, 2026
@veryCrunchy veryCrunchy added this to the 0.6 Platform polish milestone Aug 16, 2026
@veryCrunchy veryCrunchy moved this to In Progress in Nextcloud Native Roadmap Aug 16, 2026
@obiente-cloud
obiente-cloud Bot temporarily deployed to Obiente Preview / PR #393 / NC Native August 16, 2026 14:25 Destroyed
@obiente-cloud
obiente-cloud Bot temporarily deployed to Obiente Preview / PR #393 / NC Native August 16, 2026 14:30 Destroyed

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 8e942929a7

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread ui/src/commonMain/kotlin/dev/obiente/nextcloudnative/app/NextcloudNativeApp.kt Outdated
Comment thread ui/src/commonMain/kotlin/dev/obiente/nextcloudnative/app/AppsWorkspace.kt Outdated
@veryCrunchy
veryCrunchy force-pushed the fix/home-pinned-apps branch from 6d803fd to c94a792 Compare August 16, 2026 15:05
@obiente-cloud
obiente-cloud Bot temporarily deployed to Obiente Preview / PR #393 / NC Native August 16, 2026 15:05 Destroyed
@obiente-cloud
obiente-cloud Bot temporarily deployed to Obiente Preview / PR #393 / NC Native August 16, 2026 15:10 Destroyed

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: c94a79227b

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread ui/src/commonMain/kotlin/dev/obiente/nextcloudnative/app/AppsWorkspace.kt Outdated
Comment thread ui/src/commonMain/kotlin/dev/obiente/nextcloudnative/app/AppWorkspacePins.kt Outdated
@obiente-cloud
obiente-cloud Bot temporarily deployed to Obiente Preview / PR #393 / NC Native August 16, 2026 15:21 Destroyed
@obiente-cloud
obiente-cloud Bot temporarily deployed to Obiente Preview / PR #393 / NC Native August 16, 2026 15:26 Destroyed

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 1d09a929c7

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@veryCrunchy
veryCrunchy force-pushed the fix/home-pinned-apps branch from b5ec631 to 080d23a Compare August 16, 2026 15:29
@obiente-cloud
obiente-cloud Bot temporarily deployed to Obiente Preview / PR #393 / NC Native August 16, 2026 15:29 Destroyed
@obiente-cloud
obiente-cloud Bot temporarily deployed to Obiente Preview / PR #393 / NC Native August 16, 2026 15:34 Destroyed

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 080d23aa01

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread ui/src/commonMain/kotlin/dev/obiente/nextcloudnative/app/NextcloudNativeApp.kt Outdated
@veryCrunchy
veryCrunchy force-pushed the fix/home-pinned-apps branch from a796dcf to 0376d3a Compare August 16, 2026 15:37
@obiente-cloud
obiente-cloud Bot temporarily deployed to Obiente Preview / PR #393 / NC Native August 16, 2026 15:37 Destroyed
@obiente-cloud
obiente-cloud Bot temporarily deployed to Obiente Preview / PR #393 / NC Native August 16, 2026 15:42 Destroyed
@veryCrunchy
veryCrunchy force-pushed the fix/home-pinned-apps branch from 9adebbc to c5e1cb4 Compare August 16, 2026 15:50
@obiente-cloud
obiente-cloud Bot temporarily deployed to Obiente Preview / PR #393 / NC Native August 16, 2026 15:50 Destroyed
@obiente-cloud
obiente-cloud Bot temporarily deployed to Obiente Preview / PR #393 / NC Native August 16, 2026 15:52 Destroyed
@obiente-cloud
obiente-cloud Bot temporarily deployed to Obiente Preview / PR #393 / NC Native August 16, 2026 15:56 Destroyed

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: d78e1495df

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread ui/src/commonMain/kotlin/dev/obiente/nextcloudnative/app/AppsWorkspace.kt Outdated
@veryCrunchy
veryCrunchy force-pushed the fix/home-pinned-apps branch from 4d08364 to af10859 Compare August 16, 2026 16:13
@obiente-cloud
obiente-cloud Bot temporarily deployed to Obiente Preview / PR #393 / NC Native August 16, 2026 16:13 Destroyed
@obiente-cloud
obiente-cloud Bot temporarily deployed to Obiente Preview / PR #393 / NC Native August 16, 2026 16:19 Destroyed
Signed-off-by: veryCrunchy <me@verycrunchy.dev>
@obiente-cloud
obiente-cloud Bot temporarily deployed to Obiente Preview / PR #393 / NC Native August 16, 2026 18:52 Destroyed
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

@obiente-cloud
obiente-cloud Bot temporarily deployed to Obiente Preview / PR #393 / NC Native August 16, 2026 18:57 Destroyed
@veryCrunchy
veryCrunchy merged commit dcfde9b into main Aug 16, 2026
4 of 5 checks passed
@github-project-automation github-project-automation Bot moved this from In Progress to Done in Nextcloud Native Roadmap Aug 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:dynamic-apps Adaptive contracts, semantic UI, and app integrations area:ux Navigation, accessibility, responsive design, and interaction enhancement New feature or request platform:android Android-specific implementation or validation platform:desktop Desktop-specific implementation or validation priority:P1 Required for the pre-release product experience type:feature New product capability

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant