Skip to content

Confusing paragraphs about django-storage on new ecosystem page #2121

Open
@blighj

Description

@blighj

Love the new ecosystem page, saw it linked from django news.

One potential improvement to the text of the page. Down the bottom in the learn more section. There are some potential typos making the paragraphs harder to understand.
https://www.djangoproject.com/community/ecosystem/#django-storages

This package is perfect when deploying Django in situations that do include a shared filesystem. For either uploaded files from users and/or you static media if is often necessary to store files using "object storage" such as Amazon S3.

Keep in mind there are many S3 compatible object storage systems. Most cloud providers you may have an option and if not you can run your own MinIO instance. Get more information which options are supported in the django-storages docs.

My confusions:

  • that do include - I'd have thought that should be do not include?
  • you static media if is - these seem like clearer typos of you/your and if/it, I also like adding a comma as it reads to me with a break
  • cloud providers you may have an option and if not - the 'you' seems like a typo, maybe it was meant to be 'you use', sentence might be simpler with just 'have', also reads with breaks around the 'and if not' so would add commas myself normally.

My suggested edits, I'd have opened a PR, but I'm not 100% sure of the intended meaning and I never feel on firm ground with commas.

This package is perfect when deploying Django in situations that do not include a shared filesystem. For either uploaded files from users and/or your static media, it is often necessary to store files using "object storage" such as Amazon S3.

Keep in mind there are many S3 compatible object storage systems. Most cloud providers have an option, and if not, you can run your own MinIO instance. Get more information which options are supported in the django-storages docs.

https://github.com/blighj/djangoproject.com/compare/main...blighj-patch-1?diff

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions