Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Merge Release/10.9 into Master #8373

Merged
merged 89 commits into from
Sep 26, 2018
Merged

Merge Release/10.9 into Master #8373

merged 89 commits into from
Sep 26, 2018

Conversation

loremattei
Copy link
Contributor

Merge Release/10.9 into Master

mzorz and others added 30 commits September 10, 2018 18:34
@mzorz
moved AnalyticsUtils into new package org.wordpress.android.util.anal…
8c74acb 
…ytics
@mzorz
implemented broadcaste receiver to listen to ACTION_PACKAGE_FIRST_LAU…
ca7ad8f 
…NCH first time app is run
@mzorz
added Service (API < 26) and JobService (API >= 26) to wrap interacti…
a92728e 
…on with Google Play's InstallationReferrer API
@mzorz
register receiver for Intent.ACTION_PACKAGE_FIRST_LAUNCH
500c763
@mzorz
added AppPrefs flag to rely on to know whether the info has been obta…
9caa8b9 
…ined and tracked or not
@mzorz
added proper tracking of installation referrer string to Tracks
1d9b6ed
@mzorz
ending Google Play services connection properly
8c0e560
@mzorz
connection to Google PLay should be closed only after using the retri…
0250d85 
…eved parameters
@mzorz
removed unneeded predefinedvalue for INSTALLATION_REFERRER_OBTAINED i…
146d7c7 
…n Tracks
@mzorz
added a com.android.vending.INSTALL_REFERRER Broadcast Receiver and c…
aa849ab 
…onnected it to the InstallationReferrerService
@mzorz
fixed checkstyle warnings for line lengths
a8ac732
@mzorz
removed PackageFirstLaunchReceiver as it was unneeded
4bfb225
@mzorz
removed extra empty lines
7d4e91a
@mzorz
removed extra newlines and joined linebreaks
93b4956
@mzorz
removed unneeded Integer.valueOf() boxing
695974b
@mzorz
fixed typo in comment
91c4c58
@mzorz
rephrased comment to better explain whether an error can be retried
0cd3efc
@theck13
Merge pull request #8299 from wordpress-mobile/issue/8298-install-ref…
7e69256 
…errer-tracking

Installation referrer tracking
@loremattei
Bump version to alpha-128/614
2a443df
@loremattei
Bump version to 10.9-rc-2/615
e6bcd30
@loremattei
Fix version string
cdc9a36
@0nko
Use a unique name for post remote ID extra key
dda9293
@theck13
Merge pull request #8305 from wordpress-mobile/issue/8300-post-upload…
d02d420 
…-error-message

Use a unique name for post remote ID extra key
@malinajirka
Fix ContextImpl cannot be cast to WordPress
cb1d780
@oguzkocer
Merge pull request #8310 from wordpress-mobile/issue/8309-fix-change-…
c67e6ab 
…locale-context

Fix ContextImpl cannot be cast to WordPress
@theck13
Add null check for discover tag when resuming reader post list fragment
8a59c98
@theck13
Add application log message for discover tag not found when resuming …
03fbb6c 
…reader post list fragment
@0nko
Add a reference to FluxC version that has private & pending pages
54153ff
@0nko
Add string resources for pending & private pages
deb2018
@0nko
Add a new page list type
0ed8f09 
 - necessary because a list type may contain multiple page status types
0nko and others added 25 commits September 18, 2018 15:30
@0nko
Display a message when the page isn't available
49cf5a6
@0nko
Use a simpler scrolling mechanism which works more reliably
2184af8
@0nko
Move the scroll logic into a ViewModel
aae5988
@0nko
Handle a case when the pages are not loaded yet
d7fcc9c
@0nko
Optimize imports
035a7f9
@0nko
Add the missing event string
69fb52b
@planarvoid
Merge pull request #8327 from wordpress-mobile/issue/8326-page-upload…
a1c36a7 
…-notification

Issue/8326 page upload notification
@hypest
Merge pull request #8336 from wordpress-mobile/issue/8335-missing-event
df1574f 
Add the missing event string
@loremattei
10.9-rc-3 / 618 version bump
4938484
@loremattei
Update translations for 10.9 release
237690e
@loremattei
Bump version to 10.9 / 620
5c27489
@hypest
Guard against runtime exceptions while connecting
10ea09b
@hypest
Add track for installation referrer connection failed
a32f201
@malinajirka
Fix crash when showing a dialog on Google signup/login error
0c99f05
@hypest
Merge pull request #8359 from wordpress-mobile/issue/8358-securityexc…
6d0d7fc 
…eption-crash-bind-installation-referrer-service

Issue/8358 securityexception crash bind installation referrer service
@planarvoid
Add non-null check on site when opening the ActivityLogListActivity a…
87349ac 
…nd add non-null checks into the fragment
@malinajirka
Rephrase comment in LoginActivity.onGoogleSignupError method
7879832
@planarvoid
Start Pages VM when it was killed in the background
21601ad
@planarvoid
Fix ktlint issues
4d3b12b
@0nko
Simplify the page upload continuation to prevent a possible crash
9d7de5a
@0nko
Merge pull request #8365 from wordpress-mobile/fix/start_pages_vm_whe…
cf0812b 
…n_it_was_killed_in_background

Start pages vm when it was killed in background
@hypest
Merge pull request #8361 from wordpress-mobile/issue/8360
3d77e2e 
Fix crash when showing a dialog on Google signup/login error
@planarvoid
Merge pull request #8368 from wordpress-mobile/issue/8367-fix-page-up…
73fb889 
…load-crash

Simplify the page upload continuation to prevent a possible crash
@malinajirka
Merge pull request #8362 from wordpress-mobile/fix/crash_in_activity_log
2a0faca 
Fix missing site crash in ActivityLogListActivity
@loremattei
Bump version to 10.9.1/621
c4a809f
@wpmobilebot
Copy link
Contributor

3 Warnings
⚠️ PR is missing at least one label.
⚠️ PR has more than 500 lines of code changing. Consider splitting into smaller PRs if possible.
⚠️ PR is not assigned to a milestone.

Generated by 🚫 Danger

@hypest
Copy link
Contributor

hypest commented Sep 26, 2018

LGTM!

@hypest hypest merged commit 17cae8e into master Sep 26, 2018
@hypest hypest deleted the release/10.9 branch September 26, 2018 09:17
@loremattei loremattei restored the release/10.9 branch October 4, 2018 15:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
9 participants
@loremattei @wpmobilebot @hypest @mzorz @theck13 @0nko @malinajirka @oguzkocer @planarvoid