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.

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

User.modifiedTime should be optional #10221 #10376

Merged
merged 1 commit into from
Jan 8, 2024
Merged

User.modifiedTime should be optional #10221 #10376

merged 1 commit into from
Jan 8, 2024

Conversation

edloidas
Copy link
Member

@edloidas edloidas commented Jan 4, 2024

Made the modifiedTime property optional so that the type matches some predefined users, such as ANONYMOUS.

@edloidas edloidas requested a review from rymsha January 4, 2024 15:31
@edloidas edloidas linked an issue Jan 4, 2024 that may be closed by this pull request
@alansemenov alansemenov requested a review from ComLock January 8, 2024 08:43
@rymsha rymsha merged commit 883ab56 into master Jan 8, 2024
7 checks passed
@rymsha rymsha deleted the issue-10221 branch January 8, 2024 12:16
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.

User.modifiedTime should be optional in Core TS definitions
3 participants