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

added new card notification-reveal #358

Closed
wants to merge 12 commits into from
Closed

Conversation

Ajdecoder
Copy link

Description

created notification-reveal cards

Related Issue

close #268

Change Type

Select the type of change:

  • πŸ› Bug fix
  • ✨ New feature
  • πŸ“š Documentation update
  • πŸ”§ Other (describe):

Checklist

  • I’ve read and followed the contributing guidelines.
  • Code adheres to project style and standards.
  • Tests cover changes (if applicable).
  • [] Documentation updated (if required).
  • Ready for review.

image

@hari hari self-requested a review October 10, 2024 00:42
@hari
Copy link
Contributor

hari commented Oct 10, 2024

Build is failing @Ajdecoder


./animata/card/notification-reveal.stories.tsx:23:9
Type error: Type 'Element' is not assignable to type 'string'.

  21 |       {
  22 |         title: "Notification 1",
> 23 |         description: (
     |         ^
  24 |           <p>
  25 |             Payment of <span className="bold text-blue-700">$20</span> sent for book club fee
  26 |           </p>
  

@Ajdecoder Ajdecoder temporarily deployed to preview-deployment October 10, 2024 01:58 — with GitHub Actions Inactive
@Ajdecoder
Copy link
Author

@hari now it should work

Copy link

πŸš€ Preview deployed to: https://9154adfe.animata.pages.dev

@hari
Copy link
Contributor

hari commented Oct 10, 2024

@Ajdecoder This PR contains unrelated files and package changes. Please remove them.

@Ajdecoder Ajdecoder temporarily deployed to preview-deployment October 10, 2024 05:41 — with GitHub Actions Inactive
Copy link

πŸš€ Preview deployed to: https://b842419e.animata.pages.dev

@Ajdecoder Ajdecoder closed this by deleting the head repository Oct 10, 2024
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.

Notifications reveal
2 participants