Skip to content

Commit 3990efd

Browse files
author
rauldeheer
authored
Merge pull request #2 from houfio/with-result
Add effect result to destroy
2 parents 1b61175 + d40b783 commit 3990efd

File tree

2 files changed

+4
-5
lines changed

2 files changed

+4
-5
lines changed

index.d.ts

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,3 @@
11
declare module 'use-async-effect' {
2-
import { InputIdentityList } from 'react';
3-
4-
export function useAsyncEffect(effect: () => Promise<void>, destroy?: () => void, inputs?: InputIdentityList): void;
2+
export function useAsyncEffect<V>(effect: () => Promise<V>, destroy?: (result?: V) => void, inputs?: any[]): void;
53
}

index.js

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,9 @@ const { useEffect } = require('react');
22

33
module.exports.useAsyncEffect = (effect, destroy, inputs) => {
44
useEffect(() => {
5-
effect();
5+
let result;
6+
effect().then((value) => result = value);
67

7-
return destroy;
8+
return () => destroy(result);
89
}, inputs);
910
};

0 commit comments

Comments
 (0)