Closed Meriem-BM closed 3 months 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 18, 2024 8:11am |
The ArchivedQFRounds.view.tsx
file was refactored by removing the ArchivedQFRoundsMiddleBanner
component entirely. Additionally, the rendering logic for the ArchivedQFRoundsTable
was simplified to display all archived QF rounds, eliminating the previous logic that sliced the array of archived QF rounds.
File | Change Summary |
---|---|
src/.../archivedQFRounds/ArchivedQFRounds.view.tsx |
Removed ArchivedQFRoundsMiddleBanner and simplified ArchivedQFRoundsTable rendering. |
Amidst the code, a change took place,
A banner fled without a trace,
Tables now, they proudly show,
All archived rounds in a single flow.
🐰✨ The refactor done, the code’s now lean,
A wondrous sight, oh so serene. 🌟
src/components/views/archivedQFRounds/ArchivedQFRounds.view.tsx
[error] 39-39: 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.
Related to https://github.com/Giveth/giveth-dapps-v2/issues/3748#issuecomment-2159281578
Summary by CodeRabbit
Refactor
Removed
ArchivedQFRoundsMiddleBanner
component.