application-services icon indicating copy to clipboard operation
application-services copied to clipboard

Update logins enum in android-components to use the new complex structure

Open data-sync-user opened this issue 4 years ago • 1 comments

Due to a previous limitation in Uniffi we had to use the message as the enum given that errors were flat structures at the time. Now with the latest version of uniffi we can actually pass the rich errors down.

The area in question is here: https://github.com/mozilla-mobile/android-components/blob/71e124a5dbafffb800168ea615fdbc54d7398f50/components/service/sync-logins/src/main/java/mozilla/components/service/sync/logins/DefaultLoginValidationDelegate.kt#L85-L88

┆Issue is synchronized with this Jira Task ┆Story Points: 2 ┆Epic: a-s storage components (backlog) ┆Fix Versions: Release 94 ┆Sprint End Date: 2021-10-29

data-sync-user avatar Aug 18 '21 23:08 data-sync-user

I think we should wait on this one for two reasons:

  • The offending code in the description has been removed from android-components
  • I think mozilla/uniffi-rs#1077 means that if we make this change now, our error messages will be less helpful. I'd like to address that one before this.

bendk avatar Oct 25 '21 16:10 bendk

Moved to bugzilla: https://bugzilla.mozilla.org/show_bug.cgi?id=1866507

Change performed by the Move to Bugzilla add-on.

mhammond avatar Nov 24 '23 15:11 mhammond