Skip to content

Add validate_uniqueness to README.md #181

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.

Sign up for GitHub

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

Jump to bottom
Closed
ypresto wants to merge 1 commit into from
Closed

Add validate_uniqueness to README.md #181

ypresto wants to merge 1 commit into from
+1 −0

Conversation

@ypresto
Copy link

@ypresto ypresto commented Mar 17, 2020 •
edited
Loading

(NOTE: This is false by default in activerecord-import)
@coveralls
Copy link

coveralls commented Mar 17, 2020

Coverage Status

Coverage remained the same at 97.956% when pulling f67ba67 on ypresto:patch-1 into 07cee9c on activeadmin-plugins:master.
@ypresto
Copy link
Author

ypresto commented Mar 17, 2020 •
edited
Loading

Ah, it is not changeable option.
@ypresto ypresto closed this Mar 17, 2020
@ypresto
Copy link
Author

ypresto commented Mar 17, 2020 •
edited
Loading

I should disable it because my non-primary column has both presence and uniqueness validator.
@ypresto
Copy link
Author

ypresto commented Mar 19, 2020

Finally I decided to prohibit id column in input csv.

before_batch_import: ->(importer) do # https://dev.mysql.com/doc/refman/8.0/en/insert-on-duplicate.html raise 'Cannot specify id. Use subdomain for update key.' if importer.headers.keys.include?('id')
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants

@ypresto @coveralls