Skip to content

Hotfix for 2.3.1

Choose a tag to compare

View all tags
@adamyeager adamyeager released this 16 Oct 02:01
2.3.1
3f29a51

After some debugging I found that the Pushbullet API does not work with comma characters in the modified_after field. Since this field can be a double value, and in Belgium the default conversion of a double to a string contains a comma, I changed the double to string conversion code in the getPushes method.

Reproduction of the error:

  1. Change your laptop's region setting to 'Dutch (Belgium)'
  2. Use a non integer DateTime value (as I added in the unit test)
  3. Execute the GetPushesAllSince unit test
  4. You will get the error above

Done to fix the issue:
Changed the Culture of the conversion to InvariantCulture.

Assets 2
Loading