-
Notifications
You must be signed in to change notification settings - Fork 705
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
Updated template that displays the record change history. #533
base: develop
Are you sure you want to change the base?
Conversation
Thanks for PR. |
@darklow |
In order to accept your PR, you will need to implement fallback to older versions using specific version check, similar like here:
Using any if version check functions with _lt, _lte, _gt or _gte suffixes from suit_tags :https://github.com/darklow/django-suit/blob/develop/suit/templatetags/suit_tags.py#L104 |
I understand @darklow. Or is there some other special reason to maintain support? |
Thank you for making changes. Until v1.x branch is discontinued in favour of upcoming v2-dev (which I encourage to start using already) this branch promises by docs to support all Django versions from 1.4-1.10 and while keeping compatibility doesn't require too much effort I intend to keep it that way. |
Certainly @darklow, As more people began to use v2 faster the project will be complete and structured. By the way, something that I find very interesting would be the option of keeping the vertical menu on the django-suit v2 (I've read some reviews about it, but I would leave my opinion 😄). |
@gustavosdo I see many feature requests about vertical nav for v2, therefore I will implement vertical menu too similar as v1. |
@darklow will this PR be merged? I'm using django1.10.7 and faced the same problem. |
No description provided.