-
-
Notifications
You must be signed in to change notification settings - Fork 57
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
Upload organisation logo in S3. #1008
Conversation
Looks good, but I intentionally removed those two env vars from the setup. I consolidated into a single bucket defined by To have all the data in the same bucket, we need a consistent directory structure. What do you think about:
So logos would go in (if we consistently named it logo.png we could even remove the database record for the file) For the PR I am working on for project export it would be: Basemaps:
|
Sounds good, we can store data in this format |
for more information, see https://pre-commit.ci
ae9e318
to
1086c4b
Compare
Yes using I took the opportunity to fix the linting errors on the file for pre-commit. The logo always goes to |
* added basemaps and overlays buckets in the config file * added organisation logo in s3 * logo url updated in the create organisation api * remove: basemap and bucket from config file * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * fix: add forward slash to s3_path if missing * refactor: move organization logo upload to func * fix: org creation exception handling, org id rollback * refactor: organization_crud linting errors --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: spwoodcock <[email protected]>
Changes made
S3_BUCKET_NAME_BASEMAPS
andS3_BUCKET_NAME_OVERLAYS
in the config filefmtm-data
bucket in the s3.I have used
S3_DOWNLOAD_ROOT
to get the image path. @spwoodcock is it okay?Todo