Skip to content

Hotfix Lark Get Times#4321

Merged
kodiakhq[bot] merged 7 commits intomainfrom
hotfix/lark-get-times
Sep 9, 2022
Merged

Hotfix Lark Get Times#4321
kodiakhq[bot] merged 7 commits intomainfrom
hotfix/lark-get-times

Conversation

@joeauyeung
Copy link
Copy Markdown
Contributor

@joeauyeung joeauyeung commented Sep 8, 2022

What does this PR do?

This PR changes the header of the request to Lark to properly get times.

Fixes # (issue)

Environment: Staging(main branch) / Production

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • Chore (refactoring code, technical debt, workflow improvements)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How should this be tested?

  • Test A
  • Test B

Checklist

  • I haven't read the contributing guide
  • My code doesn't follow the style guidelines of this project
  • I haven't performed a self-review of my own code and corrected any misspellings
  • I haven't commented my code, particularly in hard-to-understand areas
  • I haven't checked if my PR needs changes to the documentation
  • I haven't checked if my changes generate no new warnings
  • I haven't added tests that prove my fix is effective or that my feature works
  • I haven't checked if new and existing unit tests pass locally with my changes

zomars and others added 4 commits September 7, 2022 19:09
@zomars
Removed disconnected calendar notice
638600a
@zomars @alishaz-polymath
fixes dark mode piblic page (#4201) (#4203)
47ac58a 
Co-authored-by: Syed Ali Shahbaz <52925846+alishaz-polymath@users.noreply.github.com>
@alannnc
fix cookie name (#4295)
1aefa0e
@ee07b570 @joeauyeung @PeerRich
fix: lark get free busy param error (#4285)
1ace46c 
Co-authored-by: chengcheng.frontend <chengcheng.frontend@bytedance.com>
Co-authored-by: Joe Au-Yeung <65426560+joeauyeung@users.noreply.github.com>
Co-authored-by: Peer Richelsen <peeroke@gmail.com>
@vercel
Copy link
Copy Markdown

vercel bot commented Sep 8, 2022 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
cal ✅ Ready (Inspect) Visit Preview Sep 9, 2022 at 0:52AM (UTC)

zomars
zomars approved these changes Sep 9, 2022
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.

I don't know that this is doing but approved

@joeauyeung joeauyeung added ♻️ autoupdate tells kodiak to keep this branch up-to-date automerge labels Sep 9, 2022
@kodiakhq kodiakhq bot merged commit d022582 into main Sep 9, 2022
@kodiakhq kodiakhq bot deleted the hotfix/lark-get-times branch September 9, 2022 00:43
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

automerge ♻️ autoupdate tells kodiak to keep this branch up-to-date

Projects

No open projects
Cal.com Core Roadmap
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@joeauyeung @zomars @alannnc @ee07b570