Modifications Page de contribution #3
Conversation
There was a problem hiding this comment.
Utiliser les composants shadcn/ui
S'assurer d'utiliser les composants shadcn/ui dès possible afin d'éviter de récréer la roue.
Lint, Format et Typecheck
S'assurer que le code est bien formatter.
Avant chaque PR executer ces commandes
npm run lint:fix
npm run format:write
npm run typecheckNB: Bien gérer la responsivité
Pour le nom de la page donation fera l'affaire
| handleDonationType: (value: string) => void | ||
| } | ||
|
|
||
| const DonationTypes = ({handleDonationType}:TypeProps) => { |
There was a problem hiding this comment.
Serait préférable d'utiliser des Tabs, tu peux voir sur shadcn/ui
There was a problem hiding this comment.
Si on a deux tabs on aura quasi le même contenu dans les deux tabs, raison pour laquelle j'ai évité les tabs de la library shadcn-ui
| console.log(frequency); | ||
|
|
||
| return ( | ||
| <form> |
There was a problem hiding this comment.
Pour le formulaire, utiliser les Form shadcn/ui
| subtitle: string | ||
| } | ||
|
|
||
| const TitleTile = ({ title, subtitle }: Props) => { |
There was a problem hiding this comment.
Tu pouvais utiliser le component Card de @/components/ui/card
|
Vos reviews seront pris en compte pour le prochain PR. Merci |
|
@dmd99 is attempting to deploy a commit to the codif Team on Vercel. A member of the Team first needs to authorize it. |
There was a problem hiding this comment.
Du très bon travail, j'apprécie beaucoup le temps et l'effort dispensé.
Je t'encourage à visiter ce code pour voir un exemple d'utilisation du component Form. Cela peut aider à améliorer le code:
https://github.com/Tariha/xassida.sn-fontend/blob/feature/audio/app/(admin)/dashboard/audios/components/AudioModalForm/AudioForm.tsx
config/pages.ts
Outdated
| subtitle: "<strong>Tout don que vous ferez par charité vous sera remplacé par votre Seigneur,\ | ||
| title: "Donation", | ||
| subtitle: | ||
| "<strong>Tout don que vous ferez par charité vous sera remplacé par votre Seigneur,\ |
There was a problem hiding this comment.
Un moyen de ne pas utiliser du HTML dans le texte ? C'est préférable
There was a problem hiding this comment.
Je vais me pencher sur la question pour voir la soltion qui sera la plus adaptée
| @@ -33,3 +35,16 @@ export function copyText(text: string[]) { | |||
| return clipboardCopy(toCopy) | |||
| } | |||
| } | |||
|
|
|||
| export function extractZodValuesForTypes() { | |||
There was a problem hiding this comment.
C'est pas plus facile d'énumérer les Types, vu qu'il y en a juste 2 ?
There was a problem hiding this comment.
Le type string[] n'est pas assignable au type cible [string, ...string[]]. Par conséquent la source founit ne correspondra pas à l'élement requis à la position [0] de la cible raison pour laquelle j'ai contourné le problème de cette façon.
| } from "@/components/ui/dropdown-menu" | ||
|
|
||
| interface FrequencyProps { | ||
| frequencies: IDonationFrequency[] |
There was a problem hiding this comment.
Importer les fréquences au lieu de le passer en props. Cela permettrait de rendre le component un peu plus indépendant
|
Je suis ravi que mon assistance vous soit utile. Votre retour est très apprécié. Je prends note de votre suggestion concernant le composant Form et j'irai certainement jeter un coup d'œil pour comprendre comment il est utilisé dans le code. De base je suis en front j'utilise Vuejs/Nuxt, donc contribuer à ce projet me permet de m'ameliorer sur cette technologie avec des exemples concret et des fonctionnalités avancées ainsi c'est à moi de vous remercier. N'hésitez donc surtout pas à me faire part de la moindre remarque. |
1. Config:
nous avous modifié
config/pages.tsxpour y ajouter un objet contribute qui va sotcker les variables title et subtitle denotre page contribute
2. constants:
Nous avons créer et typé plusieurs variables dans
lib/constants.ts, leurs types sonts enregistrés dans le nouveau fichiertypes/donations.ts. Ces variables nous seront utiles tout au long de la construction de notre travail et leur déclaraation centralisé nous facilitera la tache an cas de modifications3. Core modifs:
un seul fichier core a été modifier:
components/Navbar/NavigationDrawer/DonationCard.tsxafin de rendre fonctionnel le button avec next-link4. Nous avons creer le sous-répertoire suivant dans le répertoire
appDans le tout il n'y a que des composantsn front de bases qui ne perturberons pas le fonstionnement de l'application