Skip to content

Commit b69dbb9

Browse files
committed
chore(sheets): remove reviewed clipboard TODOs
1 parent a8c5a3f commit b69dbb9

2 files changed

Lines changed: 0 additions & 2 deletions

File tree

packages/sheets-ui/src/services/clipboard/__tests__/clipboard-state.service.spec.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -317,7 +317,6 @@ describe('SheetClipboardService', () => {
317317
expect(writtenHtml).toContain('</table><!--EndFragment-->');
318318
});
319319

320-
// TODO(@ai-review): Verify that Univer-owned Excel-compatible HTML never triggers the external-Excel warning on Windows.
321320
it('recognizes its Excel-compatible clipboard html as internal content on Windows', async () => {
322321
const { injector, service } = createTestContext('Windows');
323322
selectCell(injector);

packages/sheets-ui/src/services/clipboard/clipboard.service.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -378,7 +378,6 @@ export class SheetClipboardService extends Disposable implements ISheetClipboard
378378
// Firstly see if the html content is from Excel
379379
const copyId = extractId(html);
380380
const isInternalCopy = Boolean(copyId && this._copyContentCache.get(copyId));
381-
// TODO(@ai-review): Check that external Excel HTML cannot suppress the Windows shortcut warning with a stale or forged copy ID.
382381
if (this._platformService.isWindows && htmlIsFromExcel(html) && !isInternalCopy) {
383382
this._notificationService.show({
384383
type: 'warning',

0 commit comments

Comments
 (0)