Initial commit

[jitli98]

I did the file uploads on Express but I've yet to figure out how exactly I should store them in the PostgreSQL database. Will definitely look into it the next few days.

[vedantroy]

@jitli98 For inserting a file into postgres, look at this: https://dba.stackexchange.com/questions/1742/how-to-insert-file-data-into-a-postgresql-bytea-column

The mentioned postgres function (probably pg_read_file) isn't supported by knex.js. You will have to use knex.raw to manually write the necessary SQL.

For inserting files into the postgres DB, I would recommend searching for "postgres insert file into database" rather than searching about knex.js because the end result is going to be a raw SQL query that you will just execute using knex.raw(some string)

[jitli98]

Completed the integration of Postgres into file uploads.
I've also included instructions for setting up file table in the migration file.
Let me know if you want me to include those instructions in the README instead.

[vedantroy]

Completed the integration of Postgres into file uploads.
I've also included instructions for setting up file table in the migration file.
Let me know if you want me to include those instructions in the README instead.

Add the instructions for running all migrations with npm run knex migrate:latest, or whatever the command should be to the README.

[vedantroy]

I'm getting a "MulterError: Unexpected field". Do you have an idea of what the issue might be?

[jitli98]

When u do a POST request on POSTMAN, make sure to include a field name when attaching file. In this case, the field name is 'myfile'.
I'll include a screenshot of post request on Slack.

[jitli98]

Fixed the aforementioned issues.

[vedantroy]

Sorry this is taking so long. I'll be more prompt about reviewing future PRs. One more request:

• Change the route to "/files//upload". I think files/uploadfile/<user> is a bit redundant.

[jitli98]

You want me to remove the user query parameter as well?

[vedantroy]

@jitli98 Yeah. If the user is in the route, then it shouldn't have to be in a query parameter.