Skip to content

fix #1056 #1058

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.

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

Merged
merged 3 commits into from
Mar 22, 2021
Merged

fix #1056 #1058

merged 3 commits into from
Mar 22, 2021

Conversation

promer94
Copy link
Collaborator

@codesandbox-ci
Copy link

codesandbox-ci bot commented Mar 21, 2021

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 0bc51dd:

Sandbox Source
SWR-Basic Configuration
SWR-States Configuration
SWR-Infinite Configuration
swr5-concurrent-suspense-bug PR

@shuding
Copy link
Member

shuding commented Mar 21, 2021

Can you also add https://codesandbox.io/s/swr5-concurrent-suspense-bug-onut5?file=/src/App.tsx as a test case? Thanks!

Copy link
Member

@shuding shuding left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@shuding shuding merged commit 9efb6c1 into vercel:master Mar 22, 2021
@promer94 promer94 deleted the fix-suspense-loop branch March 23, 2021 00:06
@damusnet
Copy link

@shuding @promer94 @pacocoursey Hi, this issue has been fixed 3 weeks ago, yet, as far as I can tell, it still has not been published. Apparently there was briefly a v0.5.5 which was then pulled. What's the hold up? Cheers

@shuding
Copy link
Member

shuding commented Apr 12, 2021

Hi @damusnet we're aiming for the 1.0 version so new changes will be released with the "beta" tag, please try install swr@beta and this PR should be included.

@damusnet
Copy link

Thanks @shuding for the quick response.

Is there any chance this one bug fix could make it to the v0 branch? Especially if the v1 branch is going to have breaking changes, or is further out, it would be very useful for us users to have a faster way to fix this issue.

@damusnet
Copy link

edit: I see now that v0.5.5 is still available on npm, along with multiple v1 betas. The issue in my opinion is that in the absence of tagged releases with release notes here on GitHub, this is hard to follow for people like me who are not intimately familiar with the development of the project. I can't for instance just install swr@beta v1 without knowing if there are any breaking changes.

What happened with v0.5.5? Why does it not have a tag and release notes here on GitHub like previous releases?

@shuding
Copy link
Member

shuding commented Apr 12, 2021

@damusnet 0.5.5 was a hot-fix for some license related issue, and it didn't go through the normal release flow. I see your point but it's harder for the maintainers to keep tracking both versions, as patches need to be applied to both branch when developing, and there will be huge amount of conflicts when merging back potentially. I can do a patch to the 0.x version for you (let me know if you need it urgently, I can do a release tomorrow), but still encourage you to try out 1.0 (there's no breaking change in the current 1.0 beta) because new bug fixes won't be applied to 0.x eventually when 1.0 is out.

@damusnet
Copy link

Thank you for being so understanding and accommodating. I (and hopefully others) would really appreciate if you could cut a v0.x release with the usual release notes. All I need is some traceability (version points to tag points to commit hash, etc.).

@damusnet
Copy link

damusnet commented May 3, 2021

Hi @shuding

Sorry to insist, but as far as I can tell, it's been another three weeks since we last talked, and there still doesn't seem to be any non-beta release, v1.x or v0.x, that addresses the original bug fixed in this thread.

Can you please publish a v0.5.6?

shuding pushed a commit that referenced this pull request May 3, 2021
* fix #1056

* add test for #1056
@shuding
Copy link
Member

shuding commented May 3, 2021

@damusnet I just published 0.5.6.

The tricky part for us is, we have to cherry-pick this fix to the v0.5.5 commit, while we are already working on the new version for a while. We don't have 2 branches to maintain both stable and beta and make sure every fix is applied to both (and we don't have the time). In the future, we will encourage people to upgrade to the latest beta to access new features and bug fixes earlier. Otherwise they have to wait until the stable release. Thanks for your understanding!

@damusnet
Copy link

damusnet commented May 3, 2021

@shuding thanks a lot, really appreciate the new release, and totally understand the concerns!

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.

3 participants