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

[ONC-40] Reload the page if the git sha doesn't match the latest #7771

Merged
merged 2 commits into from
Mar 2, 2024

Conversation

rickyrombo
Copy link
Contributor

@rickyrombo rickyrombo commented Mar 2, 2024

ONC-40

Previous iteration used the env var, but on the rare chance that doesn't change, decided for local var instead.

@rickyrombo rickyrombo changed the title Reload the page if the git sha doesn't match the latest [ONC-40] Reload the page if the git sha doesn't match the latest Mar 2, 2024
packages/web/scripts/workers-site/index.js Show resolved Hide resolved

function* reloadDaemon() {
yield* fork(visibilityPollingDaemon, checkGitSHA())
yield* fork(foregroundPollingDaemon, checkGitSHA(), SHA_CHECK_MS)
Copy link
Member

Choose a reason for hiding this comment

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

i feel like it's enough to only check this when the app comes back into the foreground

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Probably, but with a request this cheap why not do it anyway? 😃

@audius-infra
Copy link
Collaborator

Preview this change https://demo.audius.co/mjp-reload-git-sha

Copy link
Contributor

@dylanjeffers dylanjeffers left a comment

Choose a reason for hiding this comment

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

this is dope! and it appears this only runs in web saga land right?

@rickyrombo rickyrombo enabled auto-merge (squash) March 2, 2024 01:11
@audius-infra
Copy link
Collaborator

Preview this change https://demo.audius.co/mjp-reload-git-sha

@rickyrombo rickyrombo merged commit b21f725 into main Mar 2, 2024
19 of 20 checks passed
@rickyrombo rickyrombo deleted the mjp-reload-git-sha branch March 2, 2024 01:34
audius-infra pushed a commit that referenced this pull request Mar 2, 2024
[959eac4] [PAY-2530] Add repost buttons to purchase success (#7757) Raymond Jacobson
[b21f725] [ONC-40] Reload the page if the git sha doesn't match the latest (#7771) Marcus Pasell
[871638e] Format artist tracks and followers count (#7772) Dylan Jeffers
[1ba8759] Disable play button if all collection tracks deleted on web (#7752) Saliou Diallo
rickyrombo added a commit that referenced this pull request Mar 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants