Closed Meriem-BM closed 2 weeks ago
The latest updates on your projects. Learn more about Vercel for Git โ๏ธ
Name | Status | Preview | Comments | Updated (UTC) |
---|---|---|---|---|
giveth-dapps-v2 | โ Ready (Inspect) | Visit Preview | ๐ฌ Add feedback | Jun 19, 2024 8:38am |
The changes across the codebase involve refining the donation matching messages in multiple languages, updating a HeaderWrapper component with new properties, and enhancing the EstimatedMatchingToast with additional UI logic and conditional rendering. These updates aim to clarify donation matching eligibility and improve user interaction.
Files | Change Summary |
---|---|
lang/ca.json, lang/en.json, lang/es.json | Updated and restructured donation matching toast messages |
src/components/.../ArchivedQFRounds.view.tsx | Added new properties $gap and $justifyContent to HeaderWrapper component |
src/components/.../EstimatedMatchingToast.tsx | Enhanced with new UI logic, conditional rendering, and restructuring |
sequenceDiagram
participant User
User->>DonatePage: Initiates donation
DonatePage->>EstimatedMatchingToast: Show matching toast
EstimatedMatchingToast->>ValidationLogic: Check donation amount
ValidationLogic->>EstimatedMatchingToast: Return validation result
EstimatedMatchingToast->>DonatePage: Render UI based on validation
DonatePage->>User: Display donation matching status
In the realm where code does sway,
Donation toasts now lead the way.
Languages three, they shine so bright,
With clearer texts, the goal in sight.
UI refined, logic so keen,
Together they make the perfect scene.
๐โจ
src/components/views/archivedQFRounds/ArchivedQFRounds.view.tsx
[error] 44-44: This default parameter should follow the last required parameter or should be a required parameter. (lint/style/useDefaultParameterLast) The last required parameter is here: A default parameter that precedes a required parameter cannot be omitted at call site. Unsafe fix: Turn the parameter into a required parameter.
src/components/views/archivedQFRounds/ArchivedQFRounds.view.tsx (1)
`28-30`: Addition of `$gap` and `$justifyContent` to `HeaderWrapperProps` is clear and follows TypeScript conventions.src/components/views/donate/EstimatedMatchingToast.tsx (3)
`6-6`: The addition of `IconAlertTriangleFilled` is appropriate for the warning functionality described in the PR. --- Line range hint `32-50`: Refactoring to use `React.FC` for `EstimatedMatchingToast` and the addition of new variables enhance the component's clarity and functionality. --- `53-102`: The new UI logic and conditional rendering based on `isAboveMinValidUsdValue` are well-implemented. Ensure thorough testing to confirm that the UI updates as expected.
@Ramim, for this issue the minimum USD value that @laurenluz said I had to pull from the BE was
minimumValidUsdValue
onqfRound
table, right?
Yes, but minimumValidUsdValue
in not used anywhere in the BE code, I'm not sure whether we apply this minimum amount or not.
@CarlosQ96 @mohammadranjbarz can you confirm that minimumValidUsdValue
is used in actual matching calculations or not?
@Ramim, for this issue the minimum USD value that @laurenluz said I had to pull from the BE was
minimumValidUsdValue
onqfRound
table, right?Yes, but
minimumValidUsdValue
in not used anywhere in the BE code, I'm not sure whether we apply this minimum amount or not. @CarlosQ96 @mohammadranjbarz can you confirm thatminimumValidUsdValue
is used in actual matching calculations or not?
@CarlosQ96, @mohammadranjbarz
cc: @RamRamez
minimumValidUsdValue
@RamRamez @Meriem-BM Yes we use it, you should search in the migration
folder not src
to find it, because we use it when creating the materialized view
Related to #3829
@Ramim, for this issue the minimum USD value that @laurenluz said I had to pull from the BE was
minimumValidUsdValue
onqfRound
table, right?@laurenluz should I add a message when hovering over the warning icon? like below?
Summary by CodeRabbit
New Features
Improvements
HeaderWrapper
component with new properties for better layout customization in archived QF rounds.UI Enhancements