Skip to content

Conversation

@Moamen-w-a
Copy link

@Moamen-w-a Moamen-w-a commented Dec 26, 2025

  • Update playground ContainerCode to display import when non-default transition is used
  • Add example with Bounce import in README.md
  • Fix incorrect import in useNotificationCenter.ts from 'react-toastify' to relative path
  • Update isDefaultProps to include transition check

Resolves issue #1273 : Incorrect example on documentation causing 'Bounce is not defined' error

Before submitting a pull request, please make sure the following is done:

  1. Fork the repository and create your branch from main.
  2. Run pnpm i in the repository root.
  3. If you've fixed a bug or added code that should be tested, add tests!
  4. Ensure the test suite passes (pnpm test).
  5. Run pnpm start to test your changes in the playground.
  6. Update the readme is needed
  7. Update the typescript definition is needed
  8. Format your code with prettier (pnpm prettier).
  9. Make sure your code lints (pnpm lint:fix).

For new features, please make sure that there is an issue related to it.

Learn more about contributing here

- Update playground ContainerCode to display import when non-default transition is used
- Add example with Bounce import in README.md
- Fix incorrect import in useNotificationCenter.ts from 'react-toastify' to relative path
- Update isDefaultProps to include transition check

Resolves issue fkhadra#1272: Incorrect example on documentation causing 'Bounce is not defined' error
- Changed import from 'react-toastify' to '../../index' in useNotificationCenter.ts
- Updated test import to '../../../src' for consistency
- Ensures build and tests work correctly
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