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

Fixed issue https://github.com/revsys/django-friendship/issues/86 + reformat code. #88

Closed
wants to merge 4 commits into from

Conversation

uri-rodberg
Copy link
Contributor

@uri-rodberg uri-rodberg commented Apr 25, 2019

  • Fixed issue & tests.

@uri-rodberg
Copy link
Contributor Author

  • Fixed tests.

@frankwiles frankwiles closed this May 24, 2019
@frankwiles
Copy link
Member

Appreciate the work on this and apologies that we didn't get to it sooner, but I just ran Black over the whole codebase and refactored the method before I saw this PR sadly.

@uri-rodberg
Copy link
Contributor Author

uri-rodberg commented May 25, 2019

Appreciate the work on this and apologies that we didn't get to it sooner, but I just ran Black over the whole codebase and refactored the method before I saw this PR sadly.

@frankwiles You ignored all this pull request and didn't fix other issues such as def create_user(self, username, email_address, password).

-1

@revsys revsys locked as too heated and limited conversation to collaborators May 26, 2019
@fcurella
Copy link
Collaborator

@uri-rodberg that’s not how it works.

Your PR was merged in about a month, which is IMHO is a totally acceptable time frame in open source.

I don’t know why you think you can demand first-class support for a software you’re not paying for.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants