Skip to content

Removed deprecated Tizen.Applications.AttachPanel#7702

Open
Md-OmarFaruque wants to merge 1 commit into
Samsung:mainfrom
Md-OmarFaruque:main
Open

Removed deprecated Tizen.Applications.AttachPanel#7702
Md-OmarFaruque wants to merge 1 commit into
Samsung:mainfrom
Md-OmarFaruque:main

Conversation

@Md-OmarFaruque

@Md-OmarFaruque Md-OmarFaruque commented Jun 15, 2026

Copy link
Copy Markdown

Description of Change

Removed deprecated Tizen.Applications.AttachPanel

API Changes

  • TCSACR-654

@github-actions github-actions Bot added the API15 label Jun 15, 2026

@gemini-code-assist gemini-code-assist 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.

Code Review

This pull request completely removes the Tizen.Applications.AttachPanel project and all of its associated source files, including the AttachPanel class, its internal implementations, interop definitions, event arguments, and enumerations. I have no feedback to provide.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

@JoonghyunCho

Copy link
Copy Markdown
Member

🤖 [AI Review]

Reviewed — no findings.

Scope checked:

  • All 11 AttachPanel source/interop/project files are removed; the PR-branch tree confirms no remaining paths containing attachpanel.
  • Removal is backed by ACR TCSACR-654 and is consistent with the recent DevicePolicyManager / TEEC public-API removal pattern (deprecated module retirement).
  • Packaging spec packaging/csapi-tizenfx.spec contains no AttachPanel reference, so the package build surface is not left dangling.
  • CODEOWNERS still lists the path, but the prior DevicePolicyManager/TEEC removals left their entries in place as well — this is consistent with current project practice and not a regression introduced here.

No 🔴 critical issues, no 🟡 suggestions to flag.


Automated review — final merge decision rests with human reviewers.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants