amitu / django-encrypted-id

Encrypted IDs for Django Models
BSD 2-Clause "Simplified" License
32 stars 16 forks source link

Add support for related field query (again) #11

Open Mystic-Mirage opened 7 years ago

Mystic-Mirage commented 7 years ago

Hi, guys!

I don't think you will accept this PR because it increses minimal Django version to 1.11.

This solution uses custom lookups and custom lookups for related fields only supported since Django 1.11.

So let's wait couple of years before merge :-)

amitu commented 7 years ago

Wonder what is standard solution for this kind of conundrum. May be maintain a two lines of versioning, 0.2.x without this pull request (assuming it works, I haven't tested it yet), and 0.3.x with this pull request: 0.3 only supporting Django 1.11.