search

ansys / pydynamicreporting

Pyansys project for Ansys Dynamic Reporting, a report generator tool.
https://dynamicreporting.docs.pyansys.com/
MIT License
7 stars 3 forks source link

Remove limit on template params length #108

Closed margalva closed 3 months ago

margalva commented 3 months ago

Remove limit on parameters input field. Work-around while a better implementation is designed.

margalva commented 3 months ago

@randallfrank As we're opening up the limit so now it's larger than it used to, it shouldn't pose backward compatibility issues, as far as I see it. Old cases will work with the new API. New cases won't work against old APIs, but that's not the suggested direction anyway - we don't even let you start a server from an old version on top of a new database.

viseshrp commented 3 months ago

@randallfrank I fail to see the backwards incompatibility here as it is just an extension of the number of allowed chars, as @margalva has mentioned. I also don't see the need to bump the server version due to the restrictions stated above.