search

bahnzumberg / zuugle-api

Public repositority for backend components of www.zuugle.at
https://www.zuugle.at
GNU General Public License v3.0
2 stars 3 forks source link

Is function mergeToursWithFahrplan() still needed? #23

Closed martinheppner closed 11 months ago

martinheppner commented 1 year ago

The function mergeToursWithFahrplan() is doing nothing other than updating the columns country_at, country_de, country_ch, country_it, country_fr and country_si. Is my analysis correct?

If yes, then we should deactivate it completely and remove the columns from the table as well.

Falsal commented 11 months ago

Need to check where these variables are needed , if they are really needed, will check as soon as i can

martinheppner commented 11 months ago

The answer is yes. It is partly needed. The country_xx thing has been removed, the rest is needed.