iglov / mmdb-editor

Make your own GeoIP database!
MIT License
10 stars 0 forks source link

Merge Mode and City + Org optional data #20

Closed NickyX3 closed 5 months ago

NickyX3 commented 5 months ago

Hello, i'm add merge mode (inserter.TopLevelMergeWith) by -m flag and optional City and Org import from dataset.

City is a original map, but names only "en" and "ru", Org just string

iglov commented 5 months ago

Sup bruh! Ohh, i see what do u want, but it made too awful, sry 8D Even linter shows many errors. BUT, i can add more optional keys like city, location and even postal. Let me do it 4 u ^_^

NickyX3 commented 5 months ago

Hello bro. I'm not a golang developer, I write mostly in PHP, so I don't know why the linter shows a lot of errors. go build on my windows machine works fine and exe compilation is ok, in Linux it works too but make all is an error, yes I see your new problem with these functions. Well! Let's wait. If possible, do not forget to make the merge mode, not replace, not all data usually needs to be changed, just specify the city and country

iglov commented 5 months ago

You can check new version https://github.com/iglov/mmdb-editor/releases/tag/v2.0.0 and if something is wrong, just open an issue.