novu: πŸ› Bug Report: notification center attempts to include child_process, fs, etc.

πŸ“œ Description

Module "child_process" has been externalized for browser compatibility. Cannot access "child_process.spawn" in client code.

This appears to be due to the reliance on axios which does not support esm (or at least known issues with vite).

Ref: https://github.com/vitejs/vite/issues/174, https://github.com/vitejs/vite/issues/184, etc.

Please consider using fetch instead.

πŸ‘Ÿ Reproduction steps

  1. Use vite (ts react)
  2. import { NovuProvider } from "@novu/notification-center";
  3. There ya go

πŸ‘ Expected behavior

Probably want it to work with vite.

πŸ‘Ž Actual Behavior with Screenshots

Crashes entire app.

πŸ’» Operating system

Linux

πŸ€– Node Version

16

πŸ“ƒ Provide any additional context for the Bug.

Technically running via Tauri, but I don’t think relevant to the bug

πŸ‘€ Have you spent some time to check if this bug has been raised before?

  • I checked and didn’t find similar issue

🏒 Have you read the Contributing Guidelines?

About this issue

  • Original URL
  • State: closed
  • Created 2 years ago
  • Comments: 16 (14 by maintainers)

Most upvoted comments

Yes we can close this, can be reopened again if needed πŸ™

@gitstart-novu would be great if we can use axios for now and just fix the warning.

@AlexVCS , I’ve assigned you πŸš€ , let me know if you need any help