atsign-foundation / at_widgets

Flutter widgets which aid in building applications using Atsign's technology
https://docs.atsign.com
BSD 3-Clause "New" or "Revised" License
53 stars 32 forks source link

at_login_flutter `flutter analyze` numerous warnings #767

Closed cpswan closed 8 months ago

cpswan commented 10 months ago

Which at_widget package you find issue with?

at_login_flutter

On what platforms did you experience the issue?

Linux

What version are you using?

0.0.2

What happened?

warning • The value of the field '_requestorUrl' isn't used • lib/services/at_login_service.dart:18:16 • unused_field
warning • The value of the field '_atSign' isn't used • lib/services/at_login_service.dart:20:15 • unused_field
warning • The value of the field '_nextScreen' isn't used • lib/services/at_login_service.dart:21:15 • unused_field
warning • The value of the local variable 'result' isn't used • lib/services/at_login_service.dart:62:9 • unused_local_variable
warning • The '!' will have no effect because the receiver can't be null • lib/services/custom_nav.dart:12:28 • unnecessary_non_null_assertion
warning • The '!' will have no effect because the receiver can't be null • lib/widgets/at_login_widget.dart:60:28 • unnecessary_non_null_assertion

Steps to reproduce

1. Clone this repo
2. `cd at_widgets/packages/at_login_flutter`
3. `flutter analyze --no-fatal-infos`
Sardar1208 commented 9 months ago

moving to next sprint

Sardar1208 commented 9 months ago

needs deployment and validation. Assigning 0SP and moving to next sprint

Sardar1208 commented 8 months ago

@cpswan the packages are deployed. Can we close this ticket ?