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

Fix missing fields #61

Merged
merged 5 commits into from
Aug 25, 2024
Merged

Fix missing fields #61

merged 5 commits into from
Aug 25, 2024

Conversation

aknorw
Copy link
Owner

@aknorw aknorw commented Aug 25, 2024

Introduced in #60

Ran tests locally - I need to fix CircleCI not being authorized any longer.

@@ -7,7 +7,7 @@ import { FolderIO, CollectionValueIO } from '../models/folder'

declare const client: Discojs

const rodneyfool = 'rodneyfool'
const blindborges = 'blindborges'
Copy link
Owner Author

Choose a reason for hiding this comment

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

rodneyfool changed its username 🤷‍♂️

basic_information: ReleaseMinimalInfoIO,
basic_information: ReleaseMinimalInfoIO,
}),
t.partial({
Copy link
Owner Author

Choose a reason for hiding this comment

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

These fields are not always defined.

@aknorw aknorw merged commit 112e4c7 into master Aug 25, 2024
1 check failed
@aknorw aknorw deleted the fix-missing-fields branch August 25, 2024 17:19
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.

1 participant