-
Notifications
You must be signed in to change notification settings - Fork 40
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
Add support for MySQL storage #31
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
review so far, will continue later.
{ | ||
"name": "electrode-ota-server-dao-factory", | ||
"version": "3.0.0", | ||
"description": "OTA DAO", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add author, repository, license, bugs
|
||
const nullUnlessValue = (obj, keys) => { | ||
for (const key of keys) { | ||
if (obj[key] === void(0)) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you want to null properties that do not exist also?
don't like undefined
?
I think objDefaultNull
is slightly more telling.
@@ -0,0 +1,7 @@ | |||
# THIS IS AN AUTOGENERATED FILE. DO NOT EDIT THIS FILE DIRECTLY. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can't make your mind about yarn or npm? :)
@@ -0,0 +1,23 @@ | |||
{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add author, license, repository
import _ from 'lodash'; | ||
import {Sequelize} from 'sequelize'; | ||
import {ProxyModelWrapper} from './proxy'; | ||
import cls from 'continuation-local-storage'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This. Will. Be. Your. Worst. Nightmare.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should setup eslint
Add support for MySQL/MariaDB as the data store.