jhomlala / alice

HTTP Inspector for Flutter. Allows checking HTTP connections with UI inspector.
544 stars 246 forks source link

chore: clean up examples/alice_http_client #218

Closed techouse closed 3 months ago

techouse commented 3 months ago

This PR re-applies the changes made in https://github.com/jhomlala/alice/commit/29a71dba97c7a4cabc24ab758c608fb720652687 as a PR

jhomlala commented 3 months ago

I'm back with question:

Why we're dropping support for desktop? The goal is to have support for all platforms, except web where it's easy to use build in inspector.

techouse commented 3 months ago

Does it make much sense? I think it's overkill as an example.

jhomlala commented 3 months ago

Sure, but let's imagine scenario where you're manual tester and you don't have an access to the code and want to share information about calls from some desktop application which you're testing.

techouse commented 3 months ago

Besides, it probably doesn't work because

image

So what's the point?

jhomlala commented 3 months ago

Yeah I know, it's because of the dependencies that we're using - pub.dev analyzer (pana) is really restrictive. Anyway at this moment we know that Alice works fine on mobile, but ultimate goal is to support other platforms too. Web doesn't make any sense - there are build in inspector tools in web browsers. But for desktop, we could have support without doing huge changes. At this moment it's okay to remove unsupported platforms from examples, but at the end we will revert it in some point in the future.

techouse commented 3 months ago

Ok, if that's the ultimate goal, then we add them back in the examples in the future.