Skip to content

fix: 🐛 use better mock data#4791

Merged
matteoscurati merged 1 commit intomainfrom
feat/notifications-use-better-mock-data
Oct 14, 2024
Merged

fix: 🐛 use better mock data#4791
matteoscurati merged 1 commit intomainfrom
feat/notifications-use-better-mock-data

Conversation

@matteoscurati
Copy link
Contributor

Explanation

This PR modifies some mock data returned by the NotificationServicesController to provide more realistic data for use in tests and component rendering in Storybook.

References

N/A

Changelog

@metamask/notification-services-controller

  • CHANGED: some functions in mock-raw-notifications now return more realistic mock data

Checklist

  • I've updated the test suite for new or updated code as appropriate
  • I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
  • I've highlighted breaking changes using the "BREAKING" category above as appropriate
  • I've prepared draft pull requests for clients and consumer packages to resolve any breaking changes

@matteoscurati matteoscurati requested a review from a team as a code owner October 14, 2024 08:16
@matteoscurati matteoscurati merged commit a076a7c into main Oct 14, 2024
@matteoscurati matteoscurati deleted the feat/notifications-use-better-mock-data branch October 14, 2024 10:40
matteoscurati added a commit that referenced this pull request Oct 15, 2024
## Explanation

This is a RC for v219.0.0. See changelog for more details

-  @metamask/notification-services-controller:0.10.0

## References

https://consensyssoftware.atlassian.net/browse/NOTIFY-1159

## Changelog

```
## [0.10.0]

### Changed

- update `createMockNotification` functions to provide more realistic data for use in tests and component rendering in Storybook ([#4791](#4791))

### Added

- added the ability for the `fetchFeatureAnnouncementNotifications` function, within the `notification-services-controller`, to fetch draft content from Contentful. This is made possible by passing a `previewToken` parameter ([#4790](#4790))
```

## Checklist

- [x] I've updated the test suite for new or updated code as appropriate
- [x] I've updated documentation (JSDoc, Markdown, etc.) for new or
updated code as appropriate
- [x] I've highlighted breaking changes using the "BREAKING" category
above as appropriate
- [x] I've prepared draft pull requests for clients and consumer
packages to resolve any breaking changes
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.

2 participants