Skip to content

Change request to allow more characters in select#2

Merged
langecode merged 4 commits into
openapi-tools:masterfrom
Trackunit:master
Aug 15, 2019
Merged

Change request to allow more characters in select#2
langecode merged 4 commits into
openapi-tools:masterfrom
Trackunit:master

Conversation

@gringostar
Copy link
Copy Markdown
Collaborator

@gringostar gringostar commented Jul 10, 2019

Changes:

  • Changed select logic to allow more characters in 'value' part
  • Added wildcards to select syntax, to accommodate searching by partial strings
  • Updated readme to reflect actual implementation (alternatively the implementation should be changed to allow <attribute>+/-)

@gringostar
Copy link
Copy Markdown
Collaborator Author

Hi @langecode and @AllanHoejgaardJensen

Have you had a chance to review this PR?

Best regards
Niels

Copy link
Copy Markdown
Member

@langecode langecode left a comment

Choose a reason for hiding this comment

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

I think it looks good. Thanks for the contribution and the included clean ups. I will just let @AllanHoejgaardJensen have a few more day to take a look then I will merge.

@gringostar
Copy link
Copy Markdown
Collaborator Author

I think it looks good. Thanks for the contribution and the included clean ups. I will just let @AllanHoejgaardJensen have a few more day to take a look then I will merge.

Hi
Do you think it is OK to merge this PR now?

@langecode langecode merged commit 6749c09 into openapi-tools:master Aug 15, 2019
@langecode
Copy link
Copy Markdown
Member

@gringostar I will make a release latest in the weekend. Thanks for othe contribution.

@langecode
Copy link
Copy Markdown
Member

Published as v1.0.3 - sorry for the wait.

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.

3 participants