-
Notifications
You must be signed in to change notification settings - Fork 31
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
[Tracker] SDK 4.0 - Breaking changes #1018
Comments
Thanks for creating this ticket Xavier! It's definitely time to get serious about a version 4 Re the renaming of AtKey specifically: we should consider introducing a non-breaking change much sooner by changing AtKey's implementation so that it delegates everything to a new class (likely named AtRecord), and then deprecating AtKey. We could then fully remove AtKey in a version 4 |
If we want to migrate our signatures to records, this would be a good time to do it! |
See also #1113 |
I want to consider removing KeyStreams in 4.0.0, my original intent for them doesn't really make sense given the work with AtCollections. I will need to take a closer look to confirm that they truly have no use-cases, but I believe that to be the case. |
@gkc I know we originally discussed renaming AtKey to AtID, but all of our SDKs use the term AtKey now... it's worth bringing this topic up again. |
Discussion & decision from architecture call:
|
I opened this ticket as a place to get ahead of the curve in terms of the next round of breaking changes.
The following comes to mind for this list:
Renaming AtKey will cause a breaking change, but for the benefit of making the tech easier to understand. With that, we could rename at_client_mobile to at_client_flutter, and potentially other things that we think could be made clearer.
In addition, I'm nearly certain that there are other potential changes that I am not aware of, and it would be useful to list those here as well. Then, when the time comes to actually make the breaking changes, we will have a checklist ready to ensure that we don't miss anything on release.
Other things:
Tasks
The text was updated successfully, but these errors were encountered: