I still need to work out some state update stuff, ...

# glance

[andrew]

I still need to work out some state update stuff, and maybe consider using the state definition system for some aspects, but we already have enough data duplication related tech debt that we're taking on and I'd prefer not to use that

[Summers Pittman]

The state definition system is getting reconsidered so you can lean into that preference for now.

[andrew]

Ah it is?

[Summers Pittman]

Yes (you're referring to GlanceStateDefinition right)

[andrew]

Yep 🙂

[andrew]

The datastore backed GlanceStateDefinition

[andrew]

That being said, is there any way I can ensure updates happen when I perform them? It seems like GlanceStateDefinition under the hood starts the recomposition work every time

[Summers Pittman]

Are the update methods not updating the widget?

[andrew]

Not consistently

[andrew]

Some attempts yes, others no

[Summers Pittman]

Can you share some code/example?

[andrew]

Sure 🙂

[andrew]

Which portion, the update related code?

[Summers Pittman]

Yeah, I'm trying to get enough details that I can either say "Oh that's wrong" and help you out or go "Oh that's wrong" and get the Glance side fixed

[andrew]

Screenshot 2023-06-15 at 11.25.21.png

[andrew]

In React Native, while the app is in the foreground refreshWidget will get called when access points are added to the list

[andrew]

Once it's updated + saved, a flow from a room backed db should be updating the widget

[andrew]

Screenshot 2023-06-15 at 11.27.18.png

[andrew]

Some of this code is still changing/being migrated

[Summers Pittman]

does provideGlance get called every time you call update?

[andrew]

I can check this

[Summers Pittman]

I'm trying to see if I need to check if the worker is being rate limited or if there's something wrong with the repository/state stuff. From a first glance (ha) it looks right

[andrew]

Gotcha, waiting for our test api to get back some data so I can test with (the test cluster is being a lil moody right now)

[Summers Pittman]

Sure thing! I'll do some research on my end as well. I'm really excited to see Glance+ReactNative halfway working 🙂

[andrew]

Yes! It's the next best thing for me, I usually work on Android native stuff for bluetooth and external partner integrations, and I get web dev fatigue with react native, so being able to use compose here is awesome 😎

[andrew]

Trying to push for an entire app rewrite across the org, early testing with that yields amazing benefits 🙂

[andrew]

Okay, rebuilding, had to undo a small change I made to load bitmaps from cache for initial state, was running into memory issues causing a crash

[andrew]

Seems to be fine right now... I'll keep watch of it

[andrew]

Okay, ran into it, but provideGlance was called... it must be something on react native, the joys of brownfield implementations lol

[andrew]

And it updated, just took a little bit

[andrew]

@Summers Pittman is this work manager being rate limited?