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

feat(web): enable 1 hour app delay using feature flag #7805

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

silkeholmebonnen
Copy link
Contributor

@silkeholmebonnen silkeholmebonnen commented Feb 5, 2025

Issue

GMM-406

Description

This PR uses the feature flag 1-hour-app-delay to conditionally reduce app delay to 1 hour. It adds the parameter to the backend request to get the correct data with only 1 hour delay. This only works when using the modified app backend from this PR. So this PR should not be merged until the backend PR has been merged.

Preview

Screen.Recording.2025-02-05.at.11.35.51.mov

Double check

  • I have run pnpx prettier@2 --write . and poetry run format in the top level directory to format my changes.

@silkeholmebonnen
Copy link
Contributor Author

@jbdietrich @VIKTORVAV99 Any early thoughts on this? Also do you know why the labeler is failing?

Copy link
Member

The labeler you can ignore…

Everything just seems a bit unstable since we switched to arm runners (they have better performance per watt) but if it persists we might have to revert it.

@VIKTORVAV99
Copy link
Member

@silkeholmebonnen is this ready for review or is it missing some part?

@silkeholmebonnen
Copy link
Contributor Author

@silkeholmebonnen is this ready for review or is it missing some part?

It is ready for review, but the backend PR should be merged before this one

Copy link
Member

Feel free to request my review on both if you need it!

@VIKTORVAV99 VIKTORVAV99 self-requested a review February 7, 2025 13:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants