Closed frodrigo closed 6 years ago
Thank you. Fixed.
Reintroduce French in po file
Got the process to build the required zip for JOSM.
@nlehuby I let you ask for the support of it into JOSM. For the support in Osmose I will directly use the po file.
Thanks a lot for that !
According to Dirk, the mapcss should be in the root directory, not in the data.
And we need to find a way to publish the built zip too, with a stable url.
Maybe an automated github release script can do the trick, but I'm not sure about the stable url.
Ok, I simplified the process to build the zip file.
I think we can simply commit the zip file.
I add the zip file to git to be available at download.
Rebase this PR on my new "dev" branch. This branch is now the base of my branch "osmose" https://github.com/frodrigo/transport_mapcss/tree/osmose. I already use my branch "osmose" to update into Omose the "transport" analyser. @nlehuby In Osmose I removed the two analyser in python you made and replaced it by code generated from the last mapcss of by my branch "osmose": https://github.com/frodrigo/osmose-backend/commit/188f8e4a8a0599fb3e39d4f3ae1becb440f9f69e
Ok. All done.
About Osmose: I delete python source code and move the item/class to the MapCSS generated code: https://github.com/frodrigo/osmose-backend/commit/188f8e4a8a0599fb3e39d4f3ae1becb440f9f69e#diff-511dd9d4ccd47d56b87c9b5ce8c32803L15
I moved the already done translation from Osmose Pyhton code analyser here. To help on translating this project we add an entry for it in Osmose Transifex: https://www.transifex.com/openstreetmap-france/osmose/transport_mapcss/
OK for me as it is. Github zip download not fit the need.
Hum, there is something wrong, there is no data folder in the zip anymore :thinking:
I removed because you tell me to do it, I probably misunderstood. Which is the right layout ?
I don't know ... let's return to what works:
transport_validator.zip
data
folderThere is also a regression, all the items within a group do not show up anymore ...
Do you use the group feature in Osmose ? I think it would make more sense to group all the checks from this file inside a "Public Transport" group. It would be easier to find them among other errors, and that was the reason why they were prefixed by "Jungle" in the last release.
Not to be merged. Work in progress. This branch is the a base for further work for Osmose-QA.