search

urbanairship / airship-flutter

Flutter integration
Other
17 stars 17 forks source link

[CUSTENG-3584] Add support for refreshing message center messages in flutter #70

Closed devinsmythe closed 3 years ago

devinsmythe commented 3 years ago

What do these changes do?

These changes should allow customers using the flutter SDK to manually refresh the inbox. Currently refreshes only happen when opening the app or when a push is received with a message center message.

Why are these changes necessary?

A customer wants to be able to refresh the inbox inside the app. https://urbanairship.atlassian.net/browse/CUSTENG-3584

How did you verify these changes?

I tested the changes on an android device. I have not tested the changes on iOS, I could use some help testing my iOS changes out!

Verification Screenshots:

Anything else a reviewer should know?

iOS still needs to be tested. I'm not 100% sure I called the correct iOS method.

rlepinski commented 3 years ago

Ill look into the failing CI.

The suggestions above will make it easier to implement a refresh loading indicator in the app since they will be notified with the refresh is finished using the future.

devinsmythe commented 3 years ago

@rlepinski having issues with the swift implementation. Can you take a look at what I'm doing wrong?

devinsmythe commented 3 years ago

Hopefully I got the future stuff working correctly. I tested it on android with the sample and I was able to retrieve the future value doing this so seems like it's working: Airship.refreshInbox().then((value) => print("Future value: " + value.toString()));