Closed srieteja closed 1 month ago
@srieteja I've converted this to draft until you can confirm that testing has happened. Do you need help from @CurtlyCritchlow ?
It's good to see the automated tests all green :)
I would appreciate the help, if @CurtlyCritchlow does have bandwidth available.
@srieteja Let's check off the packages as we finish manually testing.
I've tested the example apps of at_chat_flutter, at_contacts_flutter and at_contacts_group_flutter. All of these show only an onboarding_screen. (I was expecting to see chat/contact widgets on the homepage. Actual widgets might be displayed after onboarding)
Issues Observed (on Android)
Note: Since these are onboarding issues, the root cause might be lying somewhere in at_onboarding_flutter
Since this PR only contains changes to display elements, likely, the changes in this PR do not cause these issues. But we need a separate ticket to address these issues (@CurtlyCritchlow and I will discuss and create a new ticket for this)
showcaseview has released a new major version '3.0.0' during the weekend. I will go ahead and upgrade this dependency for all the packages
showcaseview has released a new major version '3.0.0' during the weekend. I will go ahead and upgrade this dependency for all the packages
@srieteja, You have to update it in at_backup_flutter. Once that package is updated and published, the issue will be resolved in all the other packages.
showcaseview has released a new major version '3.0.0' during the weekend. I will go ahead and upgrade this dependency for all the packages
@srieteja, You have to update it in at_backup_flutter. Once that package is updated and published, the issue will be resolved in all the other packages.
Yes @CurtlyCritchlow. I have upgraded it in at_backup_flutter, I'm testing all the other packages using a dep_override for at_backup_flutter
I've tested the example apps of at_chat_flutter, at_contacts_flutter and at_contacts_group_flutter. All of these show only an onboarding_screen. (I was expecting to see chat/contact widgets on the homepage. Actual widgets might be displayed after onboarding)
Issues Observed (on Android)
- at_chat_flutter: when I try to onboard/create a new atsign it just gets stuck on a loading screen
- at_contacts_flutter, at_contacts_group_flutter: when I try to fetch a new atsign, it shows an error dialogue "Unauthorized" (Possibly a missing API key or an old one).
- There are compatibility issues as the current apps are on 'compileSdkVersion 33', but during the build, I was prompted to upgrade to 'compileSdkVersion 34'. Without this change, the build process fails (Not sure if this was something related to my Android SDK setup or current Android platform requirement)
Note: Since these are onboarding issues, the root cause might be lying somewhere in at_onboarding_flutter
Since this PR only contains changes to display elements, likely, the changes in this PR do not cause these issues. But we need a separate ticket to address these issues (@CurtlyCritchlow and I will discuss and create a new ticket for this)
I was able to run at_chat_flutter, at_contacts_flutter and at_contacts_group_flutter successfully, when using an existing atsign. The problem I faced earlier was because I was trying to create a new atsign through the example widget.
Now I can certainly say that there is an issue with at_onboarding_flutter while trying to create a new atsign (will attach the logs at the end)
reg the issue mentioned with compileSdkVersion: It's a platform-related issue and from the articles/tickets I've read it's not a serious issue. This is caused if any of our dependencies have migrated to compileSdkVersion 34 and we have not. 'It's a standard practice to update/increase the 'compileSdkVersion' occasionally' Ref: https://github.com/flutter/flutter/issues/143605
[Resolved] Faced a new issue with at_location_flutter: I could successfully auth with my atKeys and see the at_location UI, but then the app threw an error-dialog saying 'You are not authorized'. I'm not sure what's causing this, but I'm currently working on looking into this. [EDIT] This error was due to the MAP-APIs API key not being set in the example app.
will be updated
@srieteja all the apps work for me. It's a green light for me.
Great work @srieteja and @CurtlyCritchlow
There's a lot going on here, and I think it's 99% good to go. Just one niggle about a(n overlooked?) dependency override.
@srieteja are you going to take care of publishing these to pub.dev now?
@cpswan, I will publish at_backupkey_flutter and at_onboarding_flutter later today, as these are key packages. @CurtlyCritchlow and I can coordinate on publishing the rest.
Closes https://github.com/atsign-foundation/at_widgets/issues/847 - What I did
- How to verify it
- Description for the changelog
fix: resolve flutter static analysis issues (beta)