Skip to content

Release 4.0.0 #443

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Release 4.0.0 #443

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
cb4cc2c
update changelog and bump 4.0.0-beta.0 version
nachocodoner Apr 24, 2025
f1566bb
update changelog
nachocodoner Apr 24, 2025
853e489
update changelog
nachocodoner Apr 24, 2025
bccd050
chore: update package versions and add package-lock.json
italojs Jun 9, 2025
d6b0540
UseFind should work like other hooks
Poyoman39 Mar 21, 2025
075e0eb
putting back useSyncEffect
italojs Jun 13, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 0 additions & 5 deletions packages/react-meteor-data/.npm/package/npm-shrinkwrap.json
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

21 changes: 11 additions & 10 deletions packages/react-meteor-data/.versions
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,29 +11,29 @@ [email protected]
[email protected]
[email protected]
[email protected]
[email protected].0
[email protected].2
[email protected]
[email protected]
[email protected]
[email protected]
[email protected].2
[email protected].3
[email protected]
[email protected]
[email protected]
[email protected].5
[email protected].6
[email protected]
[email protected]
[email protected]
[email protected]
[email protected]
local-test:react-meteor-data@3.0.4-beta.0
[email protected].5
local-test:react-meteor-data@4.0.0-beta.1
[email protected].6
[email protected]
[email protected]
[email protected].0
[email protected].1
[email protected]
[email protected]
[email protected].0
[email protected].1
[email protected]
[email protected]
[email protected]
Expand All @@ -43,17 +43,18 @@ [email protected]
[email protected]
[email protected]
[email protected]
react-meteor-data@3.0.4-beta.0
react-meteor-data@4.0.0-beta.1
[email protected]
[email protected]
[email protected]
[email protected]
[email protected]
[email protected]
[email protected].2
[email protected].3
[email protected]
[email protected]
[email protected]
[email protected]
[email protected].5
[email protected].6
[email protected]
zodern:[email protected]
5 changes: 5 additions & 0 deletions packages/react-meteor-data/CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,10 @@
# CHANGELOG

## v4.0.0-beta.0, xxx
* Breaking change: useFind describes no deps by default [PR#431](https://github.com/meteor/react-packages/pull/431)
* Fix concurrency issue with useFind [PR#419](https://github.com/meteor/react-packages/pull/419)
* Improve `useFind` and `useSubscribe` React suspense hooks [PR#420](https://github.com/meteor/react-packages/pull/429), [PR#430](https://github.com/meteor/react-packages/pull/430) and [PR#441](https://github.com/meteor/react-packages/pull/441)

## v3.0.3, 2024-12-30
* Add `useSubscribeSuspenseServer` hook to be used in SSR.

Expand Down
10 changes: 10 additions & 0 deletions packages/react-meteor-data/package-lock.json
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions packages/react-meteor-data/package.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
Package.describe({
name: 'react-meteor-data',
summary: 'React hook for reactively tracking Meteor data',
version: '3.0.6',
version: '4.0.0-beta.1',
documentation: 'README.md',
git: 'https://github.com/meteor/react-packages'
})
Expand All @@ -17,7 +17,7 @@ Package.onUse((api) => {
api.use('tracker')
api.use('ecmascript')
api.use('typescript')
api.use('zodern:[email protected]', 'server')
api.use('zodern:[email protected]')

api.mainModule('index.ts', ['client', 'server'], { lazy: true })
})
Expand Down
35 changes: 17 additions & 18 deletions packages/react-meteor-data/useFind.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -72,24 +72,23 @@ const fetchData = <T>(cursor: Mongo.Cursor<T>) => {
}

const useSyncEffect = (effect, deps) => {
const [cleanup, timeoutId] = useMemo(
() => {
const cleanup = effect();
const timeoutId = setTimeout(cleanup, 1000);
return [cleanup, timeoutId];
},
deps
);

useEffect(() => {
clearTimeout(timeoutId);

return cleanup;
}, [cleanup]);
};


const useFindClient = <T = any>(factory: () => (Mongo.Cursor<T> | undefined | null), deps: DependencyList = []) => {
const [cleanup, timeoutId] = useMemo(
() => {
const cleanup = effect();
const timeoutId = setTimeout(cleanup, 1000);
return [cleanup, timeoutId];
},
deps
);

useEffect(() => {
clearTimeout(timeoutId);

return cleanup;
}, [cleanup]);
};

const useFindClient = <T = any>(factory: () => (Mongo.Cursor<T> | undefined | null), deps: DependencyList) => {
const cursor = useMemo(() => {
// To avoid creating side effects in render, opt out
// of Tracker integration altogether.
Expand Down