Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Introduce DioInspector #16

Merged
merged 14 commits into from Aug 6, 2021

Conversation

AlexV525
Copy link
Contributor

@AlexV525 AlexV525 commented Aug 6, 2021

DioInspector, the Dio kit for flutter_ume.

Support the following features:

  • Inspect requests from the Dio instance by adding an interceptor manually.
  • Inspect requests through a request panel, support paging, requests clear.
  • Copy contents in requests.

image

Pull Request Checklist

  • I have read the UME contribution document and understand how to contribute, commit the code according to the rules. 我已阅读过 UME 贡献文档,并了解如何进行贡献,按照规则提交了代码
  • I have added the necessary comments in code to ensure that other contributors can understand the reason for the change. 我在修改中已经添加了必要的注释,以确保便于其他贡献者理解修改原因
  • The code has been formatted by dartfmt before push. 代码在提交前已经经过 dartfmt 进行了格式化
  • Change does not involve the adjustment of test cases. Or all existing and new tests are passing. 改动不涉及测试用例调整,或 example 工程与单元测试已经完全跑通

@AlexV525 AlexV525 mentioned this pull request Aug 6, 2021
10 tasks
@smileShirely smileShirely merged commit 1407caf into bytedance:develop_nullsafety Aug 6, 2021
@AlexV525 AlexV525 deleted the network-kit branch August 6, 2021 10:34
@AlexV525 AlexV525 restored the network-kit branch August 7, 2021 02:28
@AlexV525 AlexV525 deleted the network-kit branch August 7, 2021 02:28
smileShirely added a commit that referenced this pull request Aug 10, 2021
* [feat] Update documents

* [chore] Cherrypick be93269 for rm lock files

* [fix] Make flutter_coverage_badge git dependency with https url.

* [fix] Make flutter_coverage_badge git dependency with https url.

* [feat] Common FloatingWidget support custom height

* Automated report coverage

* [fix] show code plugin fix (#11)

* [fix] `CodeDisplayService` typo

* [fix] `return` does not break loop in `forEach` & argument `className` is not used

* [fix] test failed when there are more than one script's uri match the given `fileName`

* [feat] Introduce `DioInspector` (#16)

* feat: Initial commit.

* ♻️ Rename from `network` to `dio`

* 🚀 Implement pluggable

* ⚡️ Add paging for requests

* 🎨 Abstract constants

* ✨ Add `DioPluggable` registry

* ♻️ `DioPluggable` -> `DioInspector`

* 🎨 Use extensions for more pretty callers

* 📝 Address more documentations

* 🔥 Remove used packages

* 🎨 Improve imports

* 💬 `网络请求` -> `Dio Requests`

* 🎨 Apply suggestions

* ✅ Add tests

* [feat] Remove extra MaterialApp, use Overlay (#13)

* [feat] Remove extra MaterialApp, use Overlay

* [opt] Rm getx, remove unused comments

* [opt] Fix suggestions

* Automated report coverage

* [fix] syntax in example.

* [release] Release 0.2.1

* [release] Release 0.2.1

* [release] kits 0.2.1

* [chore] fix doc

Co-authored-by: smileShirely <smileShirely@users.noreply.github.com>
Co-authored-by: lpylpyleo <lpylpyleo@gmail.com>
Co-authored-by: Alex Li <github@alexv525.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants