search

bennylope / django-organizations

:couple: Multi-user accounts for Django projects
http://django-organizations.readthedocs.org/
BSD 2-Clause "Simplified" License
1.31k stars 212 forks source link

Update README.rst #275

Closed satcomjimmy closed 6 months ago

mgrdcm commented 6 months ago

Good catch! Probably don't even need to keep around the old style anymore, since nothing older than Django 3 is supported.

Maybe ditch this part altogether: https://github.com/bennylope/django-organizations/pull/275/files#diff-7b3ed02bc73dc06b7db906cf97aa91dec2b2eb21f2d92bc5caa761df5bbc168fR169-R180

And just keep the updated syntax block: https://github.com/bennylope/django-organizations/pull/275/files#diff-7b3ed02bc73dc06b7db906cf97aa91dec2b2eb21f2d92bc5caa761df5bbc168fR184-R194

satcomjimmy commented 6 months ago

Noob level: I had no idea my PR was going to go to the parent repo. I thought that was just to update my fork, however, I'm glad to drop the v1 syntax and update the request.

Good catch! Probably don't even need to keep around the old style anymore, since nothing older than Django 3 is supported.

Maybe ditch this part altogether: https://github.com/bennylope/django-organizations/pull/275/files#diff-7b3ed02bc73dc06b7db906cf97aa91dec2b2eb21f2d92bc5caa761df5bbc168fR169-R180

And just keep the updated syntax block: https://github.com/bennylope/django-organizations/pull/275/files#diff-7b3ed02bc73dc06b7db906cf97aa91dec2b2eb21f2d92bc5caa761df5bbc168fR184-R194