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.

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

Allow users to set a profile picture. #21

Open
FrazerSeymour opened this issue Feb 19, 2018 · 3 comments
Open

Allow users to set a profile picture. #21

FrazerSeymour opened this issue Feb 19, 2018 · 3 comments

Comments

@FrazerSeymour
Copy link
Contributor

Will need a UI for it, and the ability to store it in the DB or a ProtoBuf object.

Might be cool if we could automatically resize/compress it to avoid the "must be of size X" hurdle when selecting it, but still keep bandwidth low while sending the images around with the profile update messages.

Alternately, maybe the profile update object can either have the image or a hash of the current image,
which the client can use to see if their image is out of date and request the up to date one. This is more complex but prevents sending images around when not needed, or when like just the username is being updated. Some cases would contain the image by default, like if the MeshID has never been seen before, or the user has just updated their image and is sending the first update out to everyone.

@compscidr
Copy link
Contributor

I know Bangladesh was doing something clever on Yo! to address some of these issues - migth be worth reaching out to whoever was working on that to see if theres something usable there, or if you can at least start from whatever technique they were using.

@fractalic
Copy link
Contributor

Is the "must be of size X" thing a minimum (or maximum) size required by... something?

@FrazerSeymour
Copy link
Contributor Author

Sorry, on re-reading that it isn't really clear what I mean. By "must be of size X" I'm thinking of messages like "Sorry! Your file must be less than 2MB.". At which point the user is like "how on earth can I control for that on my cell phone", and they rage-uninstall our app.

I'm imaging we're going to want to keep file-size to a minimum (at least when broadcasting/storing), and that a way of doing that would be to apply compression and resize their image to be no bigger than whatever size we display it at in the app.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants