Premium support for our pure JavaScript UI components


Post by xminaya »

Hi,

I encountered the following type definition errors while using the upcoming version 7.3.0:

  1. The transferData function of RowReorder.allowCrossGridDrag does not include a Promise return type in its definition, although it does return a Promise in the actual implementation.

    1.png
    1.png (47.39 KiB) Viewed 6765 times
  2. For Store.traverseWhile, Store.query, and Model.traverse:

    The callback function invoked for each record does not accept the traversed record as an argument in the type definitions, although it is provided in the actual implementation.

    3.png
    3.png (13.84 KiB) Viewed 6765 times
    4.png
    4.png (26.61 KiB) Viewed 6765 times

Post by ahmadzaheer »

Hello,

Thanks for reporting. We will fix this: https://github.com/bryntum/support/issues/13016

Regards,
Ahmad Zaheer


Post by xminaya »

I found another type definition error in CalendarManagerStore’s configuration object. CalendarManagerStore inherits from the AjaxStore class, but the configuration object (CalendarManagerStoreConfig) is missing the lazyLoad property. The actual implementation lazily loads the data when the lazyLoad property is set.


Post by xminaya »

I have found another type definition error.

The 'lazyLoad' property in 'EventStoreConfig' and 'AssignmentStoreConfig' has incorrect type definitions. In 7.3.0, It is currently typed as

boolean | { lazyLoad: { chunkSize: number } } 

which introduces an unnecessary nested structure. Based on the current documentation, it should instead be

 boolean | { chunkSize: number }

Post by marcio »

Thanks xminaya,

I added those comments to the ticket to be fixed as well.

Best regards,
Márcio

How to ask for help? Please read our Support Policy


Post by xminaya »

Thanks marcio.

I'm having issue trying to upload attachments into the forum post. The system is failing to upload a 500kb GIF.


Post by mats »

@xminaya Add to the GItHub issue instead?


Post by xminaya »

I need to create a new forum post. I’ve identified an issue in 7.3.0 that is unrelated to type definition errors, and I need to upload test example apps (approximately 2 MB) that replicate the issue.


Post Reply