Skip to content
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

fix: installed pwa does not pick up new version, closes #298 #324

Merged
merged 2 commits into from
Apr 18, 2020

Conversation

Alexander-Bliznyuk
Copy link

No description provided.

@commit-lint
Copy link
Contributor

commit-lint bot commented Apr 14, 2020

Bug Fixes

Contributors

@

@ivelin
Copy link
Contributor

ivelin commented Apr 14, 2020

Hi @Alexander-Bliznyuk , I noticed that the version displayed in the UI is not the version of the release but the previous version. Its likely a bug in the release script. It probably updates package.json after release instead of before. Do you have time to look into it?

@Alexander-Bliznyuk
Copy link
Author

Hi @Alexander-Bliznyuk , I noticed that the version displayed in the UI is not the version of the release but the previous version. Its likely a bug in the release script. It probably updates package.json after release instead of before. Do you have time to look into it?

Hi, I will have a look.

@ivelin
Copy link
Contributor

ivelin commented Apr 14, 2020

Great, thank you!

@ivelin
Copy link
Contributor

ivelin commented Apr 14, 2020

This PR shows a conflict in src/store/index.js
Could you please submit an update that resolves it?

@Alexander-Bliznyuk
Copy link
Author

This PR shows a conflict in src/store/index.js
Could you please submit an update that resolves it?

This is due to the previous PR. Tomorrow I will fix all the issues.

@ivelin
Copy link
Contributor

ivelin commented Apr 14, 2020 via email

@ivelin
Copy link
Contributor

ivelin commented Apr 16, 2020

Good morning, @Alexander-Bliznyuk . Just checking whether you are blocked and waiting on me?

@Alexander-Bliznyuk
Copy link
Author

Sorry for the delay.
I resolved the conflict, but couldn't reproduce the problem with outdated version displayed in the UI. What steps can I go through to reveal the bug?
semantic-release is configured to push updated package.json to Github and it actually gets included in the commits alongside CHANGELOG.md.

@Alexander-Bliznyuk
Copy link
Author

Спасибо!

On Tue, Apr 14, 2020 at 12:23 PM Alexander-Bliznyuk < @.***> wrote: This PR shows a conflict in src/store/index.js Could you please submit an update that resolves it? This is due to the previous PR. Tomorrow I will fix all the issues. — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#324 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AARBUFPL7UVCVWTIKZIA3WDRMSL2PANCNFSM4MHSNGPQ .

Заповядайте!

@ivelin ivelin merged commit 247c5b9 into ambianic:master Apr 18, 2020
@ivelin
Copy link
Contributor

ivelin commented Apr 18, 2020

:)

@ivelin
Copy link
Contributor

ivelin commented Apr 18, 2020

Sorry for the delay.
I resolved the conflict, but couldn't reproduce the problem with outdated version displayed in the UI. What steps can I go through to reveal the bug?
semantic-release is configured to push updated package.json to Github and it actually gets included in the commits alongside CHANGELOG.md.

It is not obvious to me where the bug is. I also though the sem release script is fine. Maybe it was a one time issues with the git commit plugin. I will keep an eye on it and raise a flag if it persists.

@ivelin
Copy link
Contributor

ivelin commented Apr 18, 2020

🎉 This PR is included in version 2.2.9 🎉

The release is available on:

Your semantic-release bot 📦🚀

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 11, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants