[WIP] Notifications in Dashboard
PR-Codex overview
The focus of this PR is to add a notification feature to the dashboard app, displaying various types of notifications with details and timestamps.
Detailed summary
- Added
NotificationButtoncomponent to display notifications - Created
useNotificationshook to fetch fake notifications - Defined
NotificationInfotype for notifications - Implemented notification content rendering and styling
✨ Ask PR-Codex anything about this PR by commenting with
/codex {your question}
The latest updates on your projects. Learn more about Vercel for Git ↗︎
| Name | Status | Preview | Comments | Updated (UTC) |
|---|---|---|---|---|
| thirdweb_playground | ✅ Ready (Inspect) | Visit Preview | 💬 Add feedback | Jul 26, 2024 3:58pm |
| thirdweb-www | ✅ Ready (Inspect) | Visit Preview | 💬 Add feedback | Jul 26, 2024 3:58pm |
| wallet-ui | ✅ Ready (Inspect) | Visit Preview | 💬 Add feedback | Jul 26, 2024 3:58pm |
⚠️ No Changeset found
Latest commit: 581dc33ce99dba63027c450c5f40f3f3340614e4
Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.
This PR includes no changesets
When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types
Click here to learn what changesets are, and how to add one.
Click here if you're a maintainer who wants to add a changeset to this PR
Your org requires the Graphite merge queue for merging into main
Add the label “merge-queue” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge. Or use the label “hotfix” to add to the merge queue as a hot fix.
You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link.
CodSpeed Performance Report
Merging #3671 will not alter performance
Comparing mnn/notifications (581dc33) with main (ebfcbd7)
Summary
✅ 9 untouched benchmarks
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 61.50%. Comparing base (
ebfcbd7) to head (581dc33).
Additional details and impacted files
@@ Coverage Diff @@
## main #3671 +/- ##
=======================================
Coverage 61.50% 61.50%
=======================================
Files 971 971
Lines 76089 76089
Branches 3776 3776
=======================================
Hits 46801 46801
Misses 28603 28603
Partials 685 685
| Flag | Coverage Δ | *Carryforward flag | |
|---|---|---|---|
| legacy_packages | 65.68% <ø> (ø) |
Carriedforward from ebfcbd7 | |
| packages | 60.81% <ø> (ø) |
*This pull request uses carry forward flags. Click here to find out more.
size-limit report 📦
| Path | Size | Loading time (3g) | Running time (snapdragon) | Total time |
|---|---|---|---|---|
| thirdweb (esm) | 43.17 KB (0%) | 864 ms (0%) | 2.8 s (-7.86% 🔽) | 3.7 s |
| thirdweb (cjs) | 91.22 KB (0%) | 1.9 s (0%) | 5.2 s (-17.52% 🔽) | 7 s |
| thirdweb (minimal + tree-shaking) | 4.79 KB (0%) | 96 ms (0%) | 155 ms (-0.99% 🔽) | 251 ms |
| thirdweb/chains (tree-shaking) | 492 B (0%) | 10 ms (0%) | 86 ms (+116.18% 🔺) | 96 ms |
| thirdweb/react (minimal + tree-shaking) | 13.54 KB (0%) | 271 ms (0%) | 337 ms (-29.24% 🔽) | 608 ms |