js icon indicating copy to clipboard operation
js copied to clipboard

[WIP] Notifications in Dashboard

Open MananTank opened this issue 1 year ago • 6 comments

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 NotificationButton component to display notifications
  • Created useNotifications hook to fetch fake notifications
  • Defined NotificationInfo type for notifications
  • Implemented notification content rendering and styling

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

MananTank avatar Jul 12 '24 15:07 MananTank

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

vercel[bot] avatar Jul 12 '24 15:07 vercel[bot]

⚠️ 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

changeset-bot[bot] avatar Jul 12 '24 15:07 changeset-bot[bot]

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.

graphite-app[bot] avatar Jul 12 '24 15:07 graphite-app[bot]

CodSpeed Performance Report

Merging #3671 will not alter performance

Comparing mnn/notifications (581dc33) with main (ebfcbd7)

Summary

✅ 9 untouched benchmarks

codspeed-hq[bot] avatar Jul 12 '24 15:07 codspeed-hq[bot]

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.

codecov[bot] avatar Jul 12 '24 15:07 codecov[bot]

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

github-actions[bot] avatar Jul 12 '24 15:07 github-actions[bot]