-
Notifications
You must be signed in to change notification settings - Fork 6
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
Release/0.2.0 #25
Release/0.2.0 #25
Conversation
PR #22. * Add customPostPath to Snowplow class * Add docstring for customPostPath * Update Android config reader * Set customPostPath for Android * Set customPostPath for iOS * Set customPostPath for web * Restore demo app endpoint * Tidy up code * Remove leading slash, but add it back for web * Update docstring * Update docs * Allow iOS tracker to add a leading / if necessary
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚀 🚢
I had a couple of comments that I think should be addressed but I'm sure you'll handle that so approving already!
CHANGELOG
Outdated
Upgrade underlying mobile native trackers to version 4 (#17) | ||
Fix schema link in documentation for ScreenView (#12) | ||
Remove loading custom JavaScript for session context and reading cookies (#13) | ||
Upgrade min Flutter, Dart and Android SDK versions and upgrade dependencies (#19) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you please attribute this partly to @koga
who contribute part of the upgrades in PR 14 and identified the problem
@@ -1,15 +0,0 @@ | |||
# 0.1.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I understand that this changelog has a different format than the changelog in our other trackers but this is the format that is used in Dart packages and I think it is necessary for pub.dev to create a nice Changelog section.
Question: Why did you remove the alpha releases from the changelog? They were also published on pub.dev so I think it makes sense to have them here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh ok! I removed them as part of standardising with other changelogs. Will revert.
d45369e
to
0da3229
Compare
0da3229
to
f69ced8
Compare
Now this is merged, when can we expect a release on pub.dev? |
As soon as I get our release action working again 😅 hopefully today |
@erickok It's released 😊 |
@mscwilson Thank you very much! It is working very well so far, including the new custom post path. Will report if we find any glitches whatsoever. |
We are pleased to announce the release of Flutter tracker v0.2.0. It now works with Flutter 3!
We've updated the versions of the internal Snowplow trackers: it now uses JavaScript tracker v3.5 (for Web), Android v4, and iOS v4. The upgrade to JavaScript tracker v3.5 meant we were able to remove a JS file, making the tracker load more easily, and most importantly, load using Flutter 3.
The ability to set a custom POST path has also been added. Thanks to @erickok for the suggestion.
CHANGELOG
New Features
Configure custom POST path (#15)
Bug fixes
Remove loading custom JavaScript for session context and reading cookies (#13)
Under The Hood
Upgrade underlying mobile native trackers to version 4 (#17)
Fix schema link in documentation for ScreenView (#12)
Upgrade min Flutter, Dart and Android SDK versions and upgrade dependencies (#19) - thanks @koga for the Android upgrade!