43 events gallery of photos

[AvilaAndre]

Closes #43

Creates the endpoints to manage events' photos.

Review checklist

• Behavior is as expected
• Clean, well structured code
• Properly documents API changes in the corresponding controller test
• Contains enough appropriate tests
• Changes are reflected in the Wiki if necessary

[MRita443]

Would it be possible to generalize the gallery to activity? We were discussing this today and think that projects benefit from a gallery too for all the images in their details page

[AvilaAndre]

Would it be possible to generalize the gallery to activity? We were discussing this today and think that projects benefit from a gallery too for all the images in their details page

Of course! I'll get onto it :)

[github-actions]

Check the documentation preview: https://65c247bae925ce12954d3af9--niaefeup-backend-docs.netlify.app

[MRita443]

Hi @AvilaAndre are the alterations now ready?

[MRita443]

Few suggestions so far, will look at the tests after these changes :)

[github-actions]

Check the documentation preview: https://67129de7cdd15a87bce58df2--niaefeup-backend-docs.netlify.app

[MRita443]

Some things:

• Tests are missing for Project
• Tests are missing the documentation functions

Rest is mostly questions and things my IDE nagged me about :)

[github-actions]

Check the documentation preview: https://672b6109beadb96db6cad65e--niaefeup-backend-docs.netlify.app

[github-actions]

Check the documentation preview: https://672b62a7ae6d590f327f8d9e--niaefeup-backend-docs.netlify.app

[MRita443]

I was just testing this out on Postman and noticed a couple more things

The image of the event itself has an incorrect port, perhaps we should fix that in this PR too.
Also, I think maybe using the slug would be better than the title for the filename

[MRita443]

Also, regarding the documentation, I don't think it'll be possible for now because #144 is still open, my bad!

[AvilaAndre]

I was just testing this out on Postman and noticed a couple more things

The image of the event itself has an incorrect port, perhaps we should fix that in this PR too. Also, I think maybe using the slug would be better than the title for the filename

Did that event already exist on your machine?
3000 is the port that was hardcoded before. I already replaced that with 8080.

[AvilaAndre]

Also, regarding the documentation, I don't think it'll be possible for now because #144 is still open, my bad!

I already added some, the multipart is missing, maybe we just add that after #144 is finished.

Update: Already figured out why I can't use it.

[github-actions]

Check the documentation preview: https://672b6b5b000bd476a2798c62--niaefeup-backend-docs.netlify.app

[MRita443]

I was just testing this out on Postman and noticed a couple more things
The image of the event itself has an incorrect port, perhaps we should fix that in this PR too. Also, I think maybe using the slug would be better than the title for the filename

Did that event already exist on your machine? 3000 is the port that was hardcoded before. I already replaced that with 8080.

Yep you're right! Created another one and it's correct

[github-actions]

Check the documentation preview: https://672b94dfaeab8117c5de781c--niaefeup-backend-docs.netlify.app

[github-actions]

Check the documentation preview: https://672d1d2d364cc84188fab553--niaefeup-backend-docs.netlify.app

[github-actions]

Check the documentation preview: https://672d1eca338bf63f7924d3d7--niaefeup-backend-docs.netlify.app

[MRita443]

Great job! This ended up being a bit more than expected but you handled it well, good eye catching all the slug mentions too :)