Skip to content

Commit

Permalink
editing photo should now be done
Browse files Browse the repository at this point in the history
  • Loading branch information
ildyria committed Aug 27, 2024
1 parent 735d7b8 commit e37599d
Show file tree
Hide file tree
Showing 3 changed files with 31 additions and 0 deletions.
Binary file added public/blog/v6/20240827-1.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file added public/blog/v6/20240827-2.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
31 changes: 31 additions & 0 deletions src/content/post/2024-08-27-v6-edit-photo.mdx
Original file line number Diff line number Diff line change
@@ -0,0 +1,31 @@
---
publishDate: 2024-08-27T16:48:00Z
author: ildyria
title: 'Bite-size v6: Edit photos'
excerpt: Bite-size v6 is a series of small post showing the progress made on the development of the future version of Lychee.
image: /blog/v6/20240827-1.png
category: VueJS
tags:
- lychee
- vuejs
- v6
---
After taking some time off for personal reasons and some fixing minor bugs,
I finally added the endpoint to edit pictures.
As opposed to the version 4, this one will have a single API endpoint to edit all parameters.
I still need to add the ability to edit some of the exif data.

On the not-so-great news, I decided to drop the support of [dedoc/scramble](https://scramble.dedoc.co/).
![No more dedoc/scramble](/blog/v6/20240827-2.png)
It was used in the version 4 to provide easy API documentation.
Unfortunately, scramble fails at the following:

- proper software design respecting SOLID architecture ([refusal](https://github.com/dedoc/scramble/pull/91) to have proper design with interface and uses reflections instead to check if some methods are available).
- lack of static analysis such as Phpstan.

Furthermore, with version 6 we are now using [Spatie Data](https://spatie.be/docs/laravel-data/v4/introduction).
This allows us to generate [typescripts definitions](https://spatie.be/docs/typescript-transformer/v2/introduction) from those objects,
and thus ensuring stronger type compatibility between front-end and back-end.
Unfortunately Spatie Data is only supported in the pro version of Scramble, we respect the decision of romalytvynenko and remove it as we are no longer able to use it anymore.

Thoughts on how to document the API in an automated way are welcome.

0 comments on commit e37599d

Please sign in to comment.