Skip to content

fix: primaryCal is missing#17332

Merged
hbjORbj merged 1 commit into
mainfrom
fix/syntax-error
Oct 25, 2024
Merged

fix: primaryCal is missing#17332
hbjORbj merged 1 commit into
mainfrom
fix/syntax-error

Conversation

@hbjORbj
Copy link
Copy Markdown
Contributor

@hbjORbj hbjORbj commented Oct 25, 2024 •
edited
Loading

What does this PR do?

Stupid syntax error

Mandatory Tasks (DO NOT REMOVE)

  • I have self-reviewed the code (A decent size PR without self-review might be rejected).
  • N/A - I have updated the developer docs in /docs if this PR makes changes that would require a documentation change. If N/A, write N/A here and check the checkbox.
  • I confirm automated tests are in place that prove my fix is effective or that my feature works.

@keithwillcode keithwillcode added consumer core area: core, team members only labels Oct 25, 2024
@dosubot dosubot Bot added app-store area: app store, apps, calendar integrations, google calendar, outlook, lark, apple calendar 🐛 bug Something isn't working labels Oct 25, 2024
@graphite-app graphite-app Bot requested a review from a team October 25, 2024 17:01
@graphite-app
Copy link
Copy Markdown

graphite-app Bot commented Oct 25, 2024

Graphite Automations

"Add consumer team as reviewer" took an action on this PR • (10/25/24)

1 reviewer was added to this PR based on Keith Williams's automation.

zomars
zomars approved these changes Oct 25, 2024
View reviewed changes
Copy link
Copy Markdown
Contributor

@zomars zomars left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oof. That would be fatal. Nice fix!

@hbjORbj hbjORbj enabled auto-merge (squash) October 25, 2024 17:11
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Oct 25, 2024 •
edited
Loading

E2E results are ready!

@hbjORbj hbjORbj merged commit 820230d into main Oct 25, 2024
@hbjORbj hbjORbj deleted the fix/syntax-error branch October 25, 2024 17:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@zomars zomars zomars approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

app-store area: app store, apps, calendar integrations, google calendar, outlook, lark, apple calendar 🐛 bug Something isn't working consumer core area: core, team members only ready-for-e2e

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@hbjORbj @zomars @keithwillcode