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

feat(api): add handling empty metadata suggestions #3608

Merged
merged 5 commits into from
Aug 14, 2024

Conversation

JakubKida
Copy link
Contributor

@JakubKida JakubKida commented Aug 12, 2024

Throw an ERROR_CODE_EMPTY_METADATA_SUGGESTIONS error on receiving no AI suggestions for metadata.

@JakubKida JakubKida requested review from a team as code owners August 12, 2024 13:27
@CLAassistant
Copy link

CLAassistant commented Aug 12, 2024

CLA assistant check
All committers have signed the CLA.

@JakubKida JakubKida force-pushed the empty-metadata-suggestions-error branch from e45293b to 25149cc Compare August 12, 2024 15:14
@greg-in-a-box greg-in-a-box self-requested a review August 12, 2024 17:05
Copy link
Contributor

@greg-in-a-box greg-in-a-box left a comment

Choose a reason for hiding this comment

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

I almost missed the fixed time i reviewed it, looks like ERROR_CODE_EMPTY_METADATA_SUGGESTION should be plural in the .js and the test file because the constants is ERROR_CODE_EMPTY_METADATA_SUGGESTIONS

src/api/Metadata.js Outdated Show resolved Hide resolved
src/api/__tests__/Metadata.test.js Outdated Show resolved Hide resolved
.fn()
.mockResolvedValueOnce('global')
.mockResolvedValueOnce('enterprise');
metadata.getTemplates = jest.fn().mockResolvedValueOnce('global').mockResolvedValueOnce('enterprise');
Copy link
Contributor

Choose a reason for hiding this comment

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

Are those formatting changes necessary?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Prettier changed them by default, not sure if I have it set up wrong or were the latest guidelines not applied to this file

@JakubKida JakubKida force-pushed the empty-metadata-suggestions-error branch from 38e20b7 to f039507 Compare August 13, 2024 11:27
@greg-in-a-box greg-in-a-box changed the title feat: add handling empty metadata suggestions feat(api): add handling empty metadata suggestions Aug 14, 2024
@mergify mergify bot merged commit ecd4c80 into box:master Aug 14, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants