Code Monkey home page Code Monkey logo

Comments (6)

EricSimons avatar EricSimons commented on May 14, 2024

Great suggestion! As you probably know, the current way you update your profile picture is by providing a URL to an image. This was actually done intentionally to allow image upload support in the future. Specifically, here's how it would work:

  1. Have the frontend upload the image to a storage service like Amazon S3
  2. S3/etc sends back the URL to the frontend for the hosted image upon upload success
  3. Update the user's info with that image URL via the backend API

I think framework agnostic examples of this would be good to have in this repo (i.e. vanilla examples of uploading to AWS, Azure, Digital Ocean, and other storage service providers). Thoughts?

from realworld.

eko24ive avatar eko24ive commented on May 14, 2024

@EricSimons is it required to use external file storage e.g. AWS ? Is there are some caveats/restricions when using your own server ?

from realworld.

EricSimons avatar EricSimons commented on May 14, 2024

@eko24ive nope, you could technically have the upload go through the backend as well. However, it's generally not a good idea to do that because it can easily kill your server performance. Instead, if you have the client upload to a storage server that then webhooks to your server on success, you can process the pre-uploaded data in-datacenter in a non-blocking way that won't affect your API server.

from realworld.

Cameron-C-Chapman avatar Cameron-C-Chapman commented on May 14, 2024

So @EricSimons are we thinking this falls under the extra credit umbrella or is this something we want to consider for the core spec? It doesn't feel like something that fits with the core spec since it would require adding storage provider credentials to support it.

from realworld.

sandeesh avatar sandeesh commented on May 14, 2024

I think we can keep this for the next version of the spec. This isn't a simple extra credit task since this requires changes in both frontend and backend to support this. Which would need changes from 2 different parties to make this work. I don't feel that we can accommodate this with the current spec.

from realworld.

geromegrignon avatar geromegrignon commented on May 14, 2024

Closing it as it's out of the core spec.

from realworld.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.