fix(web): close overlay after successful sync

[tyler-dane]

Closes #1478

[Copilot]

Pull request overview

Addresses #1478 by ensuring the Google Calendar import completion event reliably updates “awaiting import results” flows and triggers a refetch so the UI can exit the loading state after re-auth.

Changes:

• Update useGcalSync to avoid stale state in socket event handlers by reading awaitingImportResults via a ref and triggering a fetch on successful import end.
• Adjust/add tests around the import-start/import-end socket flow and wrap socket-callback invocations in act.
• Remove an unused SVG asset and simplify GitHub issue template labels/names.

Reviewed changes

Copilot reviewed 6 out of 7 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
packages/web/src/socket/provider/SocketProvider.test.tsx	Wraps socket callback invocations in act to ensure React state updates flush in tests.
packages/web/src/socket/hooks/useGcalSync.ts	Uses a ref to read the latest “awaiting import results” state in socket callbacks; parses results; dispatches fetch trigger on import end.
packages/web/src/socket/hooks/useGcalSync.test.ts	Adds coverage for the updated import flow and edge cases (payload formats, malformed JSON).
packages/web/src/public/svg/circle.svg	Removes a previously committed SVG asset.
.github/ISSUE_TEMPLATE/config.yml	Removes emojis from contact link names.
.github/ISSUE_TEMPLATE/2-bug-report.yml	Removes emoji from template name.
.github/ISSUE_TEMPLATE/1-feature-request.yml	Removes emoji from template name and removes a “Priority” dropdown.

[Copilot]

Pull request overview

Copilot reviewed 13 out of 14 changed files in this pull request and generated 3 comments.

Comments suppressed due to low confidence (1)

packages/web/src/components/SyncEventsOverlay/SyncEventsOverlay.tsx:20

• This component now selects selectIsImportPending, but the local variable and inline comments still refer to awaitingImportResults. Renaming the variable/comments to isImportPending (or similar) would reduce confusion when reading overlay activation logic.

  const awaitingImportResults = useAppSelector(selectIsImportPending);
  const isAuthenticating = useAppSelector(selectIsAuthenticating);

  // Show overlay when:
  // - isAuthenticating: User clicked sign-in, popup is open (from auth slice)
  // - awaitingImportResults: OAuth completed, waiting on import results
  // - importing: Calendar import in progress (from sync slice)
  const shouldBeActive = isAuthenticating || awaitingImportResults || importing;