pnotify icon indicating copy to clipboard operation
pnotify copied to clipboard

Temporal fix for compile-time errors on angular apps for "pnotify": "^5.2.0"

Open samuelsonokoi opened this issue 3 years ago • 2 comments

This might not be a perfect fix but it takes care of the compile-time errors for my angular application.

node_modules/@pnotify/core/Stack.d.ts you will have to modify line 1 to import { Notice } from './';

replace line 193 to 208 with the code below swap( one: Notice, theOther: Notice, immediate: boolean, waitAfter: boolean ): Promise<unknown>;

replace line 220 with the following fire(event: string, detail: {}): void;

In @pnotify/core/index.d.ts file replace line 250 with open(immediate?: boolean): Promise<unknown>; and replace line 261 to 265 with close( immediate?: boolean, timerHide?: boolean, waitAfterward?: boolean ): Promise<unknown>;

This fix worked for "pnotify": "^5.2.0", hopefully the next update will fix these error. Ummmm you might have to redo this again if you reinstall the same package.

samuelsonokoi avatar Mar 06 '21 14:03 samuelsonokoi

Hey just wanted to +1 this issue because I'm experiencing the exact same compile time error. For additional context, we're currently using TypeScript v4.3.0 (we're also working on an Angular codebase. I know that was mentioned in #388, although I don't believe the fact that we're using Angular is relevant to this issue).

It seems like there's been an open PR to address this issue here: #398, although the changes in that PR and the changes suggested in this issue are quite different.

I believe the suggestions in this issue are more correct, because in one of the TypeScript errors about Promise (as alluded to by @samuelsonokoi above), it's specifically saying that the generic type Promise requires one type argument.

@hperrin Anything we can do to assist? Possibly open a PR with the changes suggested here?

brian-mcallister-lab49 avatar Dec 14 '21 23:12 brian-mcallister-lab49

Thank you, this fix works perfectly on "@pnotify/core": "^5.2.0" and Angular 17 project.

danaildinev avatar Apr 02 '24 16:04 danaildinev