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

add missing remap #116

Merged

Conversation

hdd
Copy link
Contributor

@hdd hdd commented Aug 29, 2016

this should be fixing #115
please have a look and let me know.

@fredrikaverpil
Copy link
Collaborator

fredrikaverpil commented Aug 29, 2016

@hdd that's excellent.

I see the equivalent for PySide was already mapped:

PySide.QtCore.QSortFilterProxyModel = PySide.QtGui.QSortFilterProxyModel

Do we need a test for these? (I'm guessing no)

@mottosso
Copy link
Owner

Thanks @hdd.

Do we need a test for these? (I'm guessing no)

This is how I envision testing for these members.

You're welcome to have a look at how to implement something like this!

@mottosso mottosso merged commit 9cd9ca0 into mottosso:master Aug 29, 2016
@hdd
Copy link
Contributor Author

hdd commented Aug 29, 2016

Cheers, I'll have a look as soon as I have time.
(always better have tests ;) )

2016-08-29 11:27 GMT+02:00 Marcus Ottosson notifications@github.com:

Merged #116 #116.


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#116 (comment), or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAYavVajWvHaWwOkiTP9LlF0ZLv5-ljGks5qkqYKgaJpZM4JvSeS
.

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