CouncilDataProject / CouncilDataProject/cdp-backend

Store a HTTPS URI in `file.uri` for files created by CDP and stored on Google cloud storage

Open
#170 0 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
Python
Stars
25
Forks
28
PR merge metrics
No merged PRs in 30d

Description

### Feature Description

_A clear and concise description of the feature you're requesting._
Instead of storing a `gs://` URI, store a `https://` URI for `file.uri`

### Use Case

_Please provide a use case to help us understand your request in context._
The frontend app can't display or download `gs://` URI. Because of this, there is an extra step to get the download URL from the `gs://` URI which increases the load of time of some of the pages that require an `https://` URI. These include the event page (URI of transcript file) and the person page (URI of the seat and person images).

### Solution

_Please describe your ideal solution._
Use `https://` URI instead of `gs://` URI for [`file.uri`](https://github.com/CouncilDataProject/cdp-backend/blob/main/cdp_backend/database/models.py#L32).
[Also look into downsampling the seat and person images](https://github.com/CouncilDataProject/cdp-frontend/pull/203#issuecomment-1047433534) to reduce the load of the person page.

### Alternatives

_Please describe any alternatives you've considered, even if you've dismissed them._

cc @JacksonMaxfield

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.