search

panosl / django-currencies

django-currencies allows you to define different currencies, and includes template tags/filters to allow easy conversion between them.
http://pypi.python.org/pypi/django-currencies
BSD 3-Clause "New" or "Revised" License
94 stars 81 forks source link

removing nonsense from app #70

Closed bashu closed 5 years ago

bashu commented 5 years ago

@panosl those changes was very project specific and I think they should be removed from the app. django-currencies should do one thing and do it well

racitup commented 5 years ago

@bashu I think it's fairly clear that you have a conflict of interest with your competing module so my suggestion would be that you leave the maintenance of this. django-currencies does do one thing and does do it well, and is intended to be feature rich. Your module is entitled simple so I suggest if you want to remove extraneous features that others find useful, you do it on your own module. Plus I find your description of nonsense insulting.

bashu commented 5 years ago

@racitup fuck off

racitup commented 5 years ago

@racitup fuck off

What a lovely person you are, clearly rational and able to have a reasoned discussion.

einarf commented 5 years ago

You are probably right that the feature is a bit specific, but in open source you got to be more diplomatic. The feature has been around for a while and people are using it. Just purging it is not the right way to go. This would be something to figure out for the next major release.

panosl commented 5 years ago

@bashu you've helped a lot a while back and @racitup is doing amazing work. I don't it's worth losing our cool over a project. This is supposed to be fun and helpful.

There's no need to speak to each other in such a way.

Since it doesn't bog down the project, we can keep it around. If it fails to keep up with the rest of it, we might remove it in the future.

Maybe have a contrib section for all the extra cool stuff that are supported.

Cheers!