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 #319 #591

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Fix #319 #591

wants to merge 3 commits into from

Conversation

StdCarrot
Copy link

  • Fixed problem:
    typemap doesn't exist error when trying to use sub-column of CompositeType in query filter
    (equivalent as get Comparator of sub-column)

  • Solving by:
    Get sub-column's type correctly.

StdCarrot and others added 3 commits December 4, 2020 15:15
- Fixed problem: 
typemap doesn't exist error when trying to use sub-column of CompositeType in select query(equivalent as get Comparator of sub-column)

- Solving by:
Get sub-column's type correctly.
@StdCarrot
Copy link
Author

@kurtmckee I wrote the unit test from #484
Please check it out.
It make correct where statement during access to attribute of composite.

@kurtmckee
Copy link
Collaborator

Thanks @StdCarrot!

@kvesteri, when you have an opportunity, would you run the CI for this PR?

@StdCarrot
Copy link
Author

I can’t. I have no permission.
The workflows are waiting for approval to run.

@kurtmckee
Copy link
Collaborator

I don't have permissions, either. I'm hopeful that the repository owner, kvesteri, will run the CI workflows. 👍 😀

@kvesteri
Copy link
Owner

@kurtmckee I added you as a collaborator for this repository

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