Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(web3mail): Add API routes for storage ability #1

Merged
merged 4 commits into from
Jul 2, 2024

Conversation

cyrilcolinet
Copy link
Collaborator

No description provided.

@cyrilcolinet cyrilcolinet added the enhancement New feature or request label Jun 28, 2024
@cyrilcolinet cyrilcolinet requested a review from Torof June 28, 2024 08:48
@cyrilcolinet cyrilcolinet self-assigned this Jun 28, 2024
Copy link
Owner

@Torof Torof left a comment

Choose a reason for hiding this comment

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

j'ai refactorisé le code pour qu'il fonctionne, qu'il soit plus accessible et lisible.

Les deux files ajoutées ça va passer mais pas le code ajouté, parceque j'ai changé des noms de fonctions , enlevé des usestate et modifié un peu les fonctions

@cyrilcolinet
Copy link
Collaborator Author

@Torof okay, si jamais tu peux juste prendre les deux fichiers dans /api et après dans les components (.tsx), tu peux facilement réutiliser la partie que j'ai ajouté.

Après si besoin, je mets à jour avec ta branch et je te résoud les conflits ?

Copy link
Owner

@Torof Torof left a comment

Choose a reason for hiding this comment

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

super, ces deux files sont nickel, je vais pouvoir intégrer facilement avec le front

@Torof
Copy link
Owner

Torof commented Jul 2, 2024

C'est tout bon, j'ai enlevé les ajouts dans les components, et acceptés les deux routers, ce sera plus facile pour éviter les bugs et continuer de maintenir et ajouter

@Torof Torof merged commit 4645e55 into web3mail Jul 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants