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.

Sign up for GitHub

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

Jump to bottom

Removed storage and adapt code #182

Merged
merged 2 commits into from
Aug 1, 2016
Merged

Removed storage and adapt code #182

merged 2 commits into from
Aug 1, 2016

Conversation

ZuluPro
Copy link
Contributor

@ZuluPro ZuluPro commented Jul 31, 2016 •
edited
Loading

  • Removed all storage
  • Adapted Storage class, get_storage and other code
  • Added docs about how to use former storages
  • Adpated unit tests
  • Made some fix
  • Remove useless settings

I personally tested all former storages from django-storages work (except S3). I made several PR about: #174, #175, #176 and #177.
I encourage everyone to add 👍 to PRs for speed their merging.

@ZuluPro ZuluPro self-assigned this Jul 31, 2016
@coveralls
Copy link

coveralls commented Jul 31, 2016 •
edited
Loading

Coverage Status

Coverage increased (+17.3%) to 91.713% when pulling 51836e6 on ZuluPro:clean_storage into 040f5d1 on django-dbbackup:master.

@coveralls
Copy link

coveralls commented Jul 31, 2016 •
edited
Loading

Coverage Status

Coverage increased (+18.4%) to 92.878% when pulling cdf3bbe on ZuluPro:clean_storage into 040f5d1 on django-dbbackup:master.

@coveralls
Copy link

coveralls commented Jul 31, 2016 •
edited
Loading

Coverage Status

Coverage increased (+19.4%) to 93.895% when pulling 0d4b7df on ZuluPro:clean_storage into 040f5d1 on django-dbbackup:master.

@ZuluPro ZuluPro mentioned this pull request Jul 31, 2016
Closed
@coveralls
Copy link

coveralls commented Jul 31, 2016 •
edited
Loading

Coverage Status

Coverage increased (+18.4%) to 94.974% when pulling 0b7f4a7 on ZuluPro:clean_storage into a4f7a17 on django-dbbackup:master.

@coveralls
Copy link

coveralls commented Aug 1, 2016 •
edited
Loading

Coverage Status

Coverage increased (+18.4%) to 94.974% when pulling 8795ebc on ZuluPro:clean_storage into a4f7a17 on django-dbbackup:master.

@ZuluPro ZuluPro merged commit 9c0ffd6 into jazzband:master Aug 1, 2016
@ZuluPro ZuluPro deleted the clean_storage branch November 10, 2018 02:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@ZuluPro ZuluPro

Labels
enhancement storage/general
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ZuluPro @coveralls