TheNewEconomy / EconomyCore

The core code powering TNE.
Other
8 stars 5 forks source link

[FEAT]: Check database version against minimum version requirement #59

Closed A248 closed 11 months ago

A248 commented 11 months ago

Feature Description

I'm glad to see another plugin documenting its database version requirement; moreover, that TNE's requirement is relatively modern, encouraging servers to update, which helps all plugin developers by enabling the latest technologies.

It would be good for SQLConnector to validate the RDMS version on startup, such as through JDBC metadata. If the user isn't running the required version, a warning could be issued or TNE could exit with an exception.

Reasoning

Novice users may not know their database version or how to check for it. TNE can make their lives slightly easier by performing this check for them. It might save you a lot of time in responding to support requests.

creatorfromhell commented 11 months ago

Feature Description

I'm glad to see another plugin documenting its database version requirement; moreover, that TNE's requirement is relatively modern, encouraging servers to update, which helps all plugin developers by enabling the latest technologies.

It would be good for SQLConnector to validate the RDMS version on startup, such as through JDBC metadata. If the user isn't running the required version, a warning could be issued or TNE could exit with an exception.

Reasoning

Novice users may not know their database version or how to check for it. TNE can make their lives slightly easier by performing this check for them. It might save you a lot of time in responding to support requests.

This has been added into the source, will be apart of the next release.