Skip to content

fix: update app details handling and sync logic in app_sync_task#11830

Merged
zhengkunwang223 merged 1 commit into
1Panel-dev:dev-v2from
HynoR:feat/v2
Feb 9, 2026
Merged

fix: update app details handling and sync logic in app_sync_task#11830
zhengkunwang223 merged 1 commit into
1Panel-dev:dev-v2from
HynoR:feat/v2

Conversation

@HynoR
Copy link
Copy Markdown
Contributor

@HynoR HynoR commented Feb 9, 2026

修复 app 同步误删 tag 的问题

@f2c-ci-robot
Copy link
Copy Markdown

f2c-ci-robot Bot commented Feb 9, 2026

Adding the "do-not-merge/release-note-label-needed" label because no release-note block was detected, please follow our release note process to remove it.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@f2c-ci-robot
Copy link
Copy Markdown

f2c-ci-robot Bot commented Feb 9, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign zhengkunwang223 for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

- Modified the logic to retain app details computed from remote sources, only falling back to existing details when necessary.
- Enhanced the synchronization process to ensure only remote resources are included in the synced app IDs for both addition and update scenarios.
@zhengkunwang223 zhengkunwang223 merged commit 635d86a into 1Panel-dev:dev-v2 Feb 9, 2026
0 of 3 checks passed
justadri pushed a commit to justadri/1Panel that referenced this pull request Feb 13, 2026
…nel-dev#11830)

- Modified the logic to retain app details computed from remote sources, only falling back to existing details when necessary.
- Enhanced the synchronization process to ensure only remote resources are included in the synced app IDs for both addition and update scenarios.
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.

2 participants