Skip to content

feat(uploads): unified upload system#92

Merged
tuturd merged 2 commits into
devfrom
feat/upload-system
Jun 24, 2026
Merged

feat(uploads): unified upload system#92
tuturd merged 2 commits into
devfrom
feat/upload-system

Conversation

@tuturd

@tuturd tuturd commented Apr 28, 2026

Copy link
Copy Markdown
Member

No description provided.

@tuturd tuturd marked this pull request as ready for review April 29, 2026 17:15

@Remi-pcn Remi-pcn left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A part les any on a l'air good. Faudra tout bien tester en dev mais à première vue comme ca ya pas de dinguerie

Ok(res, {
data: toUploadedDocumentStatus(category, latestDocument),
});
} catch (err: any) {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ya un any (au pire bypass la rule du lint si il est irremplacable)

}

Ok(res, {});
} catch (err: any) {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ici aussi

@gylfirst gylfirst left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks Good To Me, didn't test

Pour le cas des any mentionnés par @Remi-pcn , ça ne devrait pas poser de soucis, puisqu'on attend des erreurs.
vérifier quand même que ça ne pose pas de problème lors de la merge via le linter.
Passer en error: unknown devrait résoudre le soucis (cf ici).

@tuturd tuturd merged commit 8e772ac into dev Jun 24, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants