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] Added missing float type to InBBoxFilter schema #277

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

astanishevskyi
Copy link

Fix typing in InBBoxFilter and DistanceToPointFilter. The only possible values for type are ['array', 'boolean', 'integer', 'number', 'object', 'string']. But float can be used as format of number.

Refarence to Swagger Documentation Data Types

@auvipy
Copy link
Collaborator

auvipy commented Aug 31, 2022

thanks for the patch. I have started the CI

Copy link
Collaborator

@auvipy auvipy left a comment

Choose a reason for hiding this comment

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

Your commit message does not follow our commit message style guidelines:

  • missing prefix in the commit short description
    Eg: "[feature/fix/change] Action performed"

Please read our guidelines at: http://openwisp.io/docs/developer/contributing.html#commit-message-style-guidelines

@astanishevskyi astanishevskyi changed the title [BugFix] Fix schema typing [fix] Fix schema typing Aug 31, 2022
Copy link
Member

@nemesifier nemesifier left a comment

Choose a reason for hiding this comment

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

Thanks for the fix @astanishevskyi! Changing the commit message to '[fix] Added missing float type to InBBoxFilter schema` will fix the QA check.

@astanishevskyi astanishevskyi changed the title [fix] Fix schema typing [fix] Added missing float type to InBBoxFilter schema Aug 31, 2022
@astanishevskyi
Copy link
Author

@nemesisdesign thank you, fixed

@astanishevskyi astanishevskyi requested review from nemesifier and auvipy and removed request for auvipy and nemesifier September 1, 2022 11:59
@auvipy
Copy link
Collaborator

auvipy commented Sep 1, 2022

its still failing

@nemesifier
Copy link
Member

@nemesisdesign thank you, fixed

I meant to amend the commit message with the text I indicated and force push it, eg:

git commit --amend
git push origin master

@astanishevskyi
Copy link
Author

@nemesisdesign oh, sorry. Now fixed for sure

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.

None yet

3 participants