WordPress-iOS icon indicating copy to clipboard operation
WordPress-iOS copied to clipboard
verified publisher icon wordpress-mobile

Add TTL logic to remote feature flags

Open hassaanelgarem opened this issue 3 years ago • 3 comments

Description

The app already supports remote feature flags (#14799). We want to tweak the current implementation to refresh the feature flags on every app resume, but only if the TTL (24 hours) has passed.

hassaanelgarem avatar Sep 05 '22 01:09 hassaanelgarem

Just posting an note here as to what we chatted about in the call today.

We should also make sure that we request the endpoint if we are logged in. (Re-fetch the endpoint after logging in and logging out. )

enejb avatar Sep 05 '22 21:09 enejb

@enejb I am going ahead and implementing the same in Android as well.

AjeshRPai avatar Sep 06 '22 07:09 AjeshRPai

We should also make sure that we request the endpoint if we are logged in. (Re-fetch the endpoint after logging in and logging out. )

@enejb I created #19291 to track this 👍

hassaanelgarem avatar Sep 06 '22 14:09 hassaanelgarem

After some discussion here and on slack: p1664828821652799-slack-C03T04HETR8, we decided to refresh the feature flags on every launch and relaunch but without any TTL logic.

hassaanelgarem avatar Oct 07 '22 04:10 hassaanelgarem