Skip to content

Todo #21#28

Open
LetsGetCactus wants to merge 7 commits intodevfrom
dev-upload-photos
Open

Todo #21#28
LetsGetCactus wants to merge 7 commits intodevfrom
dev-upload-photos

Conversation

@LetsGetCactus
Copy link
Owner

@LetsGetCactus LetsGetCactus commented Nov 24, 2025 •
edited
Loading

  • Added Cloudinary and Multer packages
  • Created Module and Service for cloudinary
  • Modified create() and update() in apartment and user (service and controller) to upload pictures to Cloudinary and saving their public url on db
  • Modified DTOs for create and update apartments so property pictures would be in its own DTO (Needed for img selector)
  • pictures are correctly updated, created and removed from both Cloudinary and DB

@LetsGetCactus
Todo #21
1114beb 
- Added cloudinary and multer packages
- Modified create() and update() in apartment (service and controller) to upload pictures to Cloudinary and saving their public url on db
- update() adds a new pic, but does not remove older ones
@LetsGetCactus
Merge branch 'dev' into dev-upload-photos
971217d
@LetsGetCactus
- Created pictures module
9da9bc1 
- Refactor apartment to obtain pictures by pictures module as intermediary betwen CLoudinary and Apartment
@LetsGetCactus
- Created polimorfic relations between picture-user | pending-user | …
29449d3 
…aparment
@LetsGetCactus
Change to stream for create and update on picture
e8b137e
@LetsGetCactus
- Fixed findPolimorfic : was not retrieving all pics from user + apt …
1b27baf 
…(when user was owner)

- updatePolimorf now uses parallel promises for deleting and updating pis on cloudinary
@LetsGetCactus
- Fixed minor admin permissions on update apartment pictures
f01fcb8 
-Fixed pictures controller missing paramenter
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@LetsGetCactus