Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Data for MongoDB #12

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Data for MongoDB #12

wants to merge 2 commits into from

Conversation

arkka
Copy link

@arkka arkka commented Nov 10, 2016

Added wilayah-administratif-indonesia's data for MongoDB format and translation configuration for converting MySQL data to MongoDB data.

@bobbysiagian
Copy link

Good job!

Copy link
Owner

@edwardsamuel edwardsamuel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi Arkka, thanks for submitting a pull request! MongoDB format is welcomed :)
Are you familiar with Docker? If yes, would you help me to update the Dockerfile and the run.sh? So in the next run whoever run the update script, MongoDB data will be updated as well.

@@ -0,0 +1,35 @@
table "districts" do
column "id", :key, :as => :integer, :rename_to => "code"
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it would be better if you use 2 spaces indentation for Ruby code as in https://github.com/bbatsov/ruby-style-guide#spaces-indentation.

Please check http://mongify.com/getting_started.html

# Getting started
- Install mongify `gem install mongify`
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use Gemfile instead?

@ijash
Copy link

ijash commented Aug 16, 2021

untuk yang cari mongodb berbasis repo ini, saya lihat sudah di convert di https://github.com/pedox/indonesia-mongo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants