1609: Enable store import koblenz

[f1sh1918]

Short description

As a store maintainer i want to upload the csv stores for koblenz

Proposed changes

• move storeConfig to common and use it for koblenz and nuernberg
• add runConfig for project store manager creation for koblenz
• add nuernberg buildConfig as a placeholder for koblenz to get correct categories
• add TODO for getting correct buildConfig in administration. Since nuernberg and koblenz have the same categories we can use the nuernberg build config for now

Side effects

• none

Testing

1. Create a store manager for koblenz
2. login as store manager
3. Go to Akzeptanzpartner verwalten
4. Upload stores
   Find correct store file here: https://nextcloud.tuerantuer.org/index.php/apps/files/files/6468170
   But you can also use same test files as for nuernberg

Resolved issues

Fixes: #1609

[ztefanie]

• Please add an example csv of the store upload to /administration/resources/stores

• Tested and is not working, when using the csv from nextcloud i get this error: Die erforderlichen Spalten sind nicht vorhanden oder nicht in der richtigen Reihenfolge.

Whenever you are sharing a nextcloud link it only opens the landing page for me. I do not have this problem when others are sharing links, i think the correct nextcloud link is:
https://nextcloud.tuerantuer.org/index.php/apps/files/files/6518906?dir=/Projects/Digitale%20Berechtigungskarte/Stadt%20Koblenz/Akzeptanzpartner&openfile=true
Not sure what you are sharing, but it seems not to work.

[f1sh1918]

I have no clue why its not working for you.
I clicked the link you shared before and everything is working fine for me.
I attached a working video @ztefanie
I guess the links are broken from nextcloud or they changed their pattern. Some time ago it was only needed to share a link until the file number but it seems not to be enough anymore

Bildschirmaufnahme.2024-10-03.um.12.35.51.mov

I added a sample file to the repository since i don't want to upload the real csv from koblenz

[ztefanie]

Tested again, works as expected 🎉

[steffenkleinle]

Tested on firefox, works as expected

[f1sh1918]

Tested again, works as expected 🎉

can you please recheck and give me an approve if its fine @ztefanie

[ztefanie]

The columns width is not really good. I do not like the hard coded widths, there are nice approaches to handle this dynamically, see here: palantir/blueprint#1863 (comment)

If entering chars (not numbers) or nothing for lat/long i get the wrong error message "Keine passenden Geodaten gefunden! Bitte prüfen sie die Adresse." If the lat/long is invalid, it should either show a message with the info lat/long is wrong or directly get it, if adress is valid.

Special chars should not be allowed in categoryId section, i can input there "10%" and get no error in the ui, but a 500 is thrown when clicking "import stores"

postalcode should only allow numbers

i did not check every field, but the field definitively need checks

[f1sh1918]

The columns width is not really good. I do not like the hard coded widths, there are nice approaches to handle this dynamically, see here: palantir/blueprint#1863 (comment)

If entering chars (not numbers) or nothing for lat/long i get the wrong error message "Keine passenden Geodaten gefunden! Bitte prüfen sie die Adresse." If the lat/long is invalid, it should either show a message with the info lat/long is wrong or directly get it, if adress is valid.

Special chars should not be allowed in categoryId section, i can input there "10%" and get no error in the ui, but a 500 is thrown when clicking "import stores"

postalcode should only allow numbers

i did not check every field, but the field definitively need checks

Thanks for testing!

1. Dynamic column width is not part of this pr and therefore a new issue can be created. Your provided code snippet needs some additional code and adjustments to work, feel free to implement this is in a separate pr.
2. i added a check if coordinates are not empty but not valid coordinates to show an error. If there are no coordinates you have to check that your location is a correct address which is not the case for the csv file in the resources. Empty coords with a valid address should result in getting coordinates from endpoint
3. i added a regex check.
4. I also added a check even if we allow varchars in database since a postalCode could also include characters (in other countries)

[ztefanie]

Not really happy with only a frontend check, but as this is an endpoint only for logged in users, I think it is oke, but still bad practice.

[f1sh1918]

Not really happy with only a frontend check, but as this is an endpoint only for logged in users, I think it is oke, but still bad practice.

there is an open issue for that and this functionality will not be served to end users for now. So everything is addressed