Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: notification component ui #48

Draft
wants to merge 8 commits into
base: feat/react-components
Choose a base branch
from

Conversation

bluecco
Copy link
Contributor

@bluecco bluecco commented Dec 28, 2023

No description provided.

@bluecco bluecco requested review from gergold and vladutjs December 28, 2023 14:25
import { CloseIcon } from "../../icons/CloseIcon"
import { NotificationItem } from "./NotificationItem"

// TODO: discuss structure
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@vladutjs linked to the other comments, let's discuss this type too

…ications

* feat/react-components: (32 commits)
  chore: address todo
  chore: update starknetkit on ui package and export components
  chore: remove old folder
  chore: remove wrong class
  chore: update stories and remove console log
  chore: use @argent/x-multicall package
  feat: add starknetkit to ConnectedButton component
  feat: add starknetId hook and helpers
  chore(release): 1.0.29 [skip ci]
  fix: add id for argent mobile modal overlay
  chore(release): 1.0.28 [skip ci]
  fix: add id for argent mobile modal container div
  chore(release): 1.0.27 [skip ci]
  fix: add part attribute for starknetkit modal for css override
  chore(release): 1.0.26 [skip ci]
  fix: don't show modal if only one connector is defined
  chore(release): 1.0.25 [skip ci]
  fix: rpc urls for mainnet
  chore: removed console log
  fix: allow custom provider for argent mobile connector
  ...
@bluecco bluecco marked this pull request as draft January 24, 2024 08:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant