highcharts / highcharts-ios

iOS wrapper for Highcharts.
Other
127 stars 39 forks source link

v8.2.0 - What happened to HIHour etc. and Xaxis.dateTimeLabelFormats ??? #337

Closed skuske closed 4 years ago

skuske commented 4 years ago

The latest release causes trouble with HIHour and Xaxis.dateTimeLabelFormats:

Bildschirmfoto 2020-09-10 um 10 20 33 Bildschirmfoto 2020-09-10 um 10 20 42

How do I set Xaxis dateTimeLabelFormats in v8.2.0?

It's a pain that fundamental things are changed without any information and without any documentation.

skuske commented 4 years ago

@ihnatmoisieiev

Could you please at least confirm that this is a bug or if we are missing something as we are now trying to get that working again for 5 hours. Thanks.

ihnatmoisieiev commented 4 years ago

Hello @skuske,

thank you for the reporting.

Please do not update to the latest release if these changes are destructive for you at the moment. I have reported it and will let you know what happened.

According to the newest JS API, HIHour, HIDay, etc. objects have been changed to string type. So, e.g. it's not needed to create a new HIDay object and set main property. Just set day property with string value.

About axis.dateTimeLabelFormats, I will let you know as soon as possible.

skuske commented 4 years ago

@ihnatmoisieiev

Thanks for the update. I found out about HIHour etc. and that it now expects NSString. However, the missing axis.dateTimeLabelFormats is really critical. Hopefully you get that fixed asap as we wanted to use v8.2.0 due to the fixed memory issues.

sebastianbochan commented 4 years ago

Hi @skuske, Thank you for reporting.

Due to unsuspected changes in the Highcharts core, some options disappeared. We are still debugging the problem and will fix it as soon as possible.

Thank you for your patience.

skuske commented 4 years ago

Any update on this? Thanks! :o)

pawelfus commented 4 years ago

You can check the status in the linked PR, right above your post :) Right now, fix is awaiting review & merge.

ihnatmoisieiev commented 4 years ago

@skuske axis.dateTimeLabelFormats property has been reverted in the latest release.

skuske commented 4 years ago

Fixed. Thanks! :o)