Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

feat: use rfc for timestamp cell to serde #347

Merged
merged 2 commits into from
Dec 10, 2024
Merged

feat: use rfc for timestamp cell to serde #347

merged 2 commits into from
Dec 10, 2024

Conversation

speed2exe
Copy link
Contributor

No description provided.

@speed2exe speed2exe marked this pull request as ready for review December 9, 2024 01:16
@speed2exe speed2exe marked this pull request as draft December 9, 2024 07:38
@speed2exe speed2exe marked this pull request as ready for review December 9, 2024 10:04
richardshiue
richardshiue approved these changes Dec 9, 2024
View reviewed changes
Copy link
Contributor

@richardshiue richardshiue left a comment

Choose a reason for hiding this comment

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

LGTM!

@appflowy appflowy merged commit b54a377 into main Dec 10, 2024
6 checks passed
@appflowy appflowy deleted the feat/timestamp-serde-rfc3339 branch December 10, 2024 03:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@richardshiue richardshiue richardshiue approved these changes

@appflowy appflowy Awaiting requested review from appflowy

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@speed2exe @richardshiue @appflowy