search

urbanairship / airship-flutter

Flutter integration
Other
17 stars 18 forks source link

[MOBILE-3858] Add support for feature flag #174

Closed Ulrico972 closed 1 year ago

Ulrico972 commented 1 year ago

What do these changes do?

Add support for feature flag

Why are these changes necessary?

To support feature flag

How did you verify these changes?

Manual testing

Verification Screenshots:

Anything else a reviewer should know?

rlepinski commented 1 year ago

I updated it with my feedback since ill be out. Here is the usage:

Airship.featureFlagManager.flag("another_rad_flag").then((value) => {
    debugPrint("flag: $value")
}).catchError((error) => {
    debugPrint("flag error: $error")
});
Ulrico972 commented 1 year ago

We have an issue on iOS, the proxy has a typo in one feature flag key. I think ryan found it but I'm not sure how he planed to handle it. The best way would be to fix the proxy but it might be used already in other frameworks. So I'm checking them to see what's best. Maybe I'll find the workaround in one of them.

EDIT: I'm finally fixing the proxy since it's just react that is using it for now and feature flag is not implemented yet.