Skip to content

feat(calendar): add calendar aliases (supersedes #104)#393

Merged
steipete merged 9 commits intosteipete:mainfrom
salmonumbrella:fix/pr104-calendar-alias-ci
Mar 8, 2026
Merged

feat(calendar): add calendar aliases (supersedes #104)#393
steipete merged 9 commits intosteipete:mainfrom
salmonumbrella:fix/pr104-calendar-alias-ci

Conversation

@salmonumbrella
Copy link
Contributor

Supersedes #104.

This continues @zacharytamas's calendar alias work and incorporates CI/lint compatibility updates so it passes current checks.

What’s included

  • calendar alias config storage (calendar_aliases)
  • gog calendar alias set/list/unset commands
  • alias resolution integrated across calendar commands (create/update/delete/search/specialized commands)
  • lint/format updates required by current repo rules

Verification

  • make ci passes locally
  • CGO_ENABLED=1 go build ./cmd/gog passes locally

Thanks to @zacharytamas for the original implementation.

@salmonumbrella
Copy link
Contributor Author

salmonumbrella commented Mar 3, 2026

This PR supersedes #104 and carries forward @zacharytamas's calendar alias implementation with CI/lint updates for current main rules. @visionik

@visionik
Copy link
Collaborator

visionik commented Mar 3, 2026

Thank you @salmonumbrella and @zacharytamas (for the OG).

I need one day to think about the pattern of

gog calendar alias
gog docs alias
etc

vs

gog alias alias_name any_google_identifier

Any thoughts?

@salmonumbrella
Copy link
Contributor Author

First one gog calendar work --etc

zacharytamas and others added 9 commits March 8, 2026 17:23
Add support for storing calendar aliases in config.json, following the
same pattern as account aliases. Includes CRUD functions and tests.
Add gog calendar alias {list,set,unset} commands for managing calendar
aliases. These allow mapping friendly names like 'family' to full
calendar IDs like '3656f8abc123@group.calendar.google.com'.
Add resolveCalendarID helper function and register CalendarAliasCmd
in the calendar command tree.
Update CalendarCreateCmd, CalendarUpdateCmd, CalendarDeleteCmd,
CalendarProposeTimeCmd, and CalendarRespondCmd to resolve calendar
aliases before using the calendar ID.
Update focus-time, out-of-office, working-location, and time commands
to resolve calendar aliases.
Update search, freebusy, and conflicts commands to resolve calendar
aliases. Multi-calendar commands now resolve each calendar ID in the
comma-separated list.
@steipete steipete force-pushed the fix/pr104-calendar-alias-ci branch from 3e2b78f to fe57116 Compare March 8, 2026 17:33
@steipete steipete merged commit 905c0e4 into steipete:main Mar 8, 2026
2 of 5 checks passed
@steipete
Copy link
Owner

steipete commented Mar 8, 2026

Landed on main in merge commit 905c0e4 after rebase/conflict resolution and CI fixes.

Included branch commits:

Also added a regression test for alias-backed calendar search resolution. Thanks @salmonumbrella.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants