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

Allow to overwrite the template tag's name to avoid conflicts. #161

Closed
wants to merge 1 commit into from

Conversation

philippeowagner
Copy link

Avoid conflicts with other tags for example the switch/case template tag.

This can be done by defining WAFFLE_FLAG_TAG_NAME, WAFFLE_SWITCH_TAG_NAME and/or WAFFLE_SAMPLE_TAG_NAME in the settings.py. This is all optional.

Avoid conflicts with other tags for example the switch/case template tag.

This can be done by defining ``WAFFLE_FLAG_TAG_NAME``, ``WAFFLE_SWITCH_TAG_NAME`` and/or ``WAFFLE_SAMPLE_TAG_NAME`` in the ``settings.py``. This is all optional.
@philippeowagner
Copy link
Author

Hi @jsocol et al.
Please either merge, specify issues or comment on this PR. Would be great to have this in the main project and have it released soon. We have the conflict mentioned in several project in production. :-/ Thanks.

@philippeowagner
Copy link
Author

@jsocol – any updates on this?

@michaelkuty
Copy link
Member

+1

another maintainer which has no time to manage own OS makes me sad @jsocol

@philippeowagner
Copy link
Author

@michaelkuty me too
@jsocol please check out https://jazzband.co/

@michaelkuty
Copy link
Member

@philippeowagner yes, i know, good point @jsocol jazzband is one of recommended way..

@jsocol
Copy link
Collaborator

jsocol commented Feb 9, 2016

Honestly, I think the right solution is to namespace these tags, e.g. waffle_flag / waffle_switch / waffle_sample, not to add additional code to make them customizable. (Which isn't just what's here, but would need to include tests.)

@jsocol jsocol closed this Feb 9, 2016
@philippeowagner
Copy link
Author

Renaming the template tags? Seriously? IMHO this is absolutely not the right solution either. This would break backwards compatibility and force everybody using django-waffle to rename each and every switch, sample and flag used. Agreed, the tests are missing. I'm pretty sure that I would have found the time to add some during the last 8 month...

@jsocol
Copy link
Collaborator

jsocol commented Feb 10, 2016

Waffle is still a pre-1.0 project and there's a decent amount of stabilizing to do to get over the 1.0 boundary. Breaking backwards compatibility isn't ideal, but these are badly named and better to fix it pre-1.0 than post.

On Feb 9, 2016, 7:25 PM -0500, Philippe O. Wagnernotifications@github.com, wrote:

Renaming the template tags? Seriously? IMHO this is absolutely not the right solution either. This would break backwards compatibility and force everybody using django-waffle to rename each and everyswitch,sampleandflagused. Agreed, the tests are missing. I'm pretty sure that I would have found the time to add some during the last 8 month...


Reply to this email directly orview it on GitHub(#161 (comment)).

@michaelkuty
Copy link
Member

@philippeowagner ok if author has this opinion, we can create friendly fork for us, stable and long term usable with django-comptat, I making integration waffle to other systems and I don't want refactoring all occurrences in next year.. internally I called this attitude: non invasive contribution, it's sad, but sometimes we have no choice

@jsocol
Copy link
Collaborator

jsocol commented Feb 10, 2016

I recognize that you two disagree with this, and that's fine. You are free to fork, but as an over-committed maintainer, I'd encourage you to look at Gutter, which seems to have some traction as well.

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.

None yet

3 participants