-
Notifications
You must be signed in to change notification settings - Fork 14
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
Images #62
Comments
The units for height/width in expo/RN aren't pixels. They're dimensionless, which puts an extra complication on how big we need to save or serve images. TODO: Decide on size/quality images to serve for in-feed/fullscreen/thumbnail/avatar scenarios |
Cloudinary has a couple of ways to index an entire folder
|
TODO: Figure out how captions/photo credit work. Can we/should we store these in image metadata? in the database? |
Proposed file structure (undescore/dash/camel naming conventions TBD)
*1 - store path, return index of subfolders/images *3 - How far do we want to break down the folders for feed images?
|
Does anyone know of a great RN image viewer control that behaves like FB? |
The post creation part of all this is going to be a nightmare |
Leaving this open until @shortnd confirms folder structure is in a place where we are all happy with it |
@galenriley thoughts on the folder structures that has already been in uses.. they are close to what you had above but some have been slightly modified. |
We have only been using the feed images so far, and one item in channels.
Feed works very well.
In theory, we'll have player photo albums coming in the app eventually,
though that may be shuffled in favor of using phone local storage to cache
data. Are y'all using player photos widely? (I have an album on imgur that
I reference by editing the database directly. Was waiting on the go-ahead
to use the admin tools for this purpose.)
What are y'all doing with images so far? We're using very little of our
credit balance with only the feed.
…-G
On Thu, Jun 4, 2020, 3:46 PM Collin OConnell ***@***.***> wrote:
@galenriley <https://github.com/galenriley> thoughts on the folder
structures that has already been in uses.. they are close to what you had
above but some have been slightly modified.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#62 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAEFIB5LLEPGBNZPD3VTJ5TRU72YBANCNFSM4KOVBZRA>
.
|
No description provided.
The text was updated successfully, but these errors were encountered: