NoMore201 / googleplay-api

Google Play Unofficial Python API
Other
410 stars 208 forks source link

Adds much more devices by checking out alldevices branch from play-store-api repo #81

Closed petrblaho closed 5 years ago

petrblaho commented 5 years ago

At the time of writing this PR there is an 'alldevices' branch of play-store-api which contains much more devices' properties. See https://github.com/yeriomin/play-store-api/tree/alldevices/src/main/resources

This PR modifies the fetch_devices.sh script to allow checking out different branch and updates device.properties file.

petrblaho commented 5 years ago

I did not tested imported devices though...

The properties are generated by code from YalpStore - https://github.com/yeriomin/YalpStore/blob/master/app/src/main/java/com/github/yeriomin/yalpstore/bugreport/BugReportDeviceInfoBuilder.java as mentioned in https://github.com/yeriomin/play-store-api/issues/26#issuecomment-416988667 so I assume that these properties are safe for use.

petrblaho commented 5 years ago

I tested it with https://github.com/NoMore201/googleplay-api/blob/master/test/test.py

After a lot of failings with configparser I updated fetch_devices.sh script so it ignores some issues with properties files from https://github.com/yeriomin/play-store-api/tree/alldevices/src/main/resources (missing device name, chopped off file, file from forwarded email content :-), etc)

I can log in and run test/test.py with 'bacon' or with 'addison' (the reason I started this :-))

NoMore201 commented 5 years ago

I think there are a lot of downsides with this:

  1. I don't like huge text files, since parsing it increases initialization time, and maintaining it is painful (the only exception is googleplay_pb2.py)
  2. Too many untested devices

In my opinion we should only introduce tested devices definitions via PR. Moreover we need a way to let the library load a custom device properties file so that people may test new devices without modifying the existing configuration

petrblaho commented 5 years ago

I agree with you in both points.

Thanks.

I am closing this now :-)