Feature/images #93
Loading…
Reference in a new issue
No description provided.
Delete branch "feature/images"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Fixes #20
Closes !98
current concerns:
urls.pyshould be better organized.added 11 commits
developCompare with previous version
added 2 commits
Compare with previous version
This should return
ImageRetrieveSerializer(instance=instance).data, right?Use
serializer.is_valid(raise_exception=True)for simplicity.Why call the app
imageeverywhere - except here?Do you mean the
create? or theget_serializer_class?changed this line in version 4 of the diff
changed this line in version 4 of the diff
changed this line in version 4 of the diff
added 1 commit
Compare with previous version
I mean
create. The note is always related to the line directly above the comment, Line 177 in this case.added 69 commits
developCompare with previous version
@david Would you please take a look at the integration in the
RecipeEditFormcomponent?marked the checklist item Separation between apps is pretty loose, maybe the base
urls.pyshould be better organized. as completedmarked the checklist item Frontend part (Component for upload) as completed
added 2 commits
Compare with previous version
added 1 commit
Compare with previous version
@root Here you go.
added 1 commit
Compare with previous version
added 1 commit
Compare with previous version
marked the checklist item Testing as completed
added 1 commit
Compare with previous version
added 1 commit
Compare with previous version
mentioned in merge request !98
changed the description
changed milestone to %1
mentioned in commit
be3492a9d2mentioned in commit 72eff5e50eee9a13960cb3cc5435f2c287eb949f
added 1 commit
Compare with previous version
added 1 commit
Compare with previous version
mentioned in commit 59f4a1eec6905d1022803afe6116fa323501371f
added 26 commits
developCompare with previous version
@david Did you added the possibility to view uploaded photos in the edit view? Should we change the serializer or make a request for every image with the given id?
added 23 commits
Compare with previous version
Okay. Well I screwed up one of the rebases.🙈Currently saving of recipes is broken due to the change in the serializer. Not sure how you want to handle that. Use a different serializer for saving?
mentioned in commit
1fad722f12added 29 commits
develop654101d6- Added redis rq dependency807a9adf- Add initial app, viewsets and serializers for photosf38789e8- Add resizing for uploaded images1f8e56fb- Fix colorspace and the fit of thumbnails, add media to .gitignorec7b24cdd- Fix urls in DEBUG mode and serve media files via django9e474763- Fix migration, remove debug, add compose file references for worker and redis5aff2ff1- Add initial upload support70eb2957- Add preview thumbnails and retry button if single uploads failb633ac20- Let serializer raise an exception, rename apps, create image folder if it doesn't existd9464654- Emit image id's of succesfull images9e8ed9c7- Fix formatting and sortinga5a08dd1- Add saving of images89a8e464- Fix unit testsdb8f1de1- Add persistence for media and images directory71d4faa4- Add tests for image upload and resizing3bfff0ff- Add requirements for image testsfa41c6ec- downgrade redis for testing1fad722f- Revert "Remove dependency on !93"5edd9b03- Add initial gallery components951d6150- Use ImageSerializers on recipe.a4e2c6d8- Complete gallery and image frame setupf78f7343- Add gallery to recipe detail pagei46cc9438- Add permission for image uploadCompare with previous version
added 3 commits
dbd7de85- Added image field to serializers1da2aa71- Added persistent image view and upload in frontendd53d8dcd- Remove image in frontend by clicking on themCompare with previous version
added 2 commits
150b01f5- Fix removal of images in backende0a604c6- Simplify saving of images on a recipeCompare with previous version
added 1 commit
f6a9834e- Fit images with real delete button and remove debugCompare with previous version
unmarked as a Work In Progress
@david Are you willing to have a look at this mess?
Propably somethings missing. But that should'nt be vital for the mission. 😃
added 1 commit
15b085c7- Linting and whitespace fixes in RecipeDetail viewCompare with previous version
added 1 commit
b7d81c27- More formatting issuesCompare with previous version
marked the checklist item Exif data should be removed on upload. as completed
marked the checklist item Meta information (e.g. a description on model) as completed
added 1 commit
6e6a9b17- Added missing translations for image uploadCompare with previous version
added 1 commit
1d0cd9cb- Add extra serializer for the retrieval of Recipes.Compare with previous version
resolved all discussions
mentioned in commit
832bbdc9b7merged