Skip to content

Fix address tag resolution #56

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

Merged
merged 2 commits into from
Nov 22, 2021
Merged

Conversation

scottatron
Copy link
Contributor

Tags returned from the API do not have a 'tag' attribute—the tag value is returned in the 'type' attribute.

This introduces a special case for tag lookups to allow the filter_by value to be overridden.

@cmeissner
Copy link
Member

@scottatron great work. We'll merge it asap. May I keen to ask you to open a bug too to follow the process. It will help others to understand what the issue is, how it can be reproduced and what versions are affected.
In a second step we can link the issue to that PR and last it will be great if you also add a changelog fragment under changelogs/fragments to ensure that your change is mentioned correctly in the next version changelog.

@cmeissner cmeissner self-requested a review November 18, 2021 10:05
@cmeissner cmeissner added the bug Something isn't working label Nov 18, 2021
@scottatron
Copy link
Contributor Author

Ah sorry, I should have read the CONTRIBUTING guide a little more closely. I'll get an issue raised and update the changelogs/fragments

@scottatron
Copy link
Contributor Author

This change addresses #57

Tags returned from the API do not have a 'tag' attribute—the tag value
is returned in the 'type' attribute.

This introduces a special case for tag lookups to allow the filter_by
value to be overridden.

Fixes codeaffen#57
Copy link
Member

@cmeissner cmeissner left a comment

Choose a reason for hiding this comment

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

Great work. Thanks for contributing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Specifying a tag for an address fails with phpypam.core.exceptions.PHPyPAMInvalidSyntax: Invalid filter_by error
2 participants