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

Take NPM package off TODO! :) #14

Closed
assertchris opened this issue Aug 30, 2010 · 6 comments
Closed

Take NPM package off TODO! :) #14

assertchris opened this issue Aug 30, 2010 · 6 comments

Comments

@assertchris
Copy link

assertchris commented Aug 30, 2010

No description provided.

@sdepold
Copy link
Member

sdepold commented Aug 30, 2010

I thought about removing the most of the readme and inserting a link to sequelizejs.com instead. what do you think?

@assertchris
Copy link
Author

assertchris commented Aug 30, 2010

I think that would save you a lot of work. I have been tinkering with sequelize the whole night and it works quite well. There is one thing I'm battling with at the moment, but I will open an issue ticket for it... :)

@sdepold
Copy link
Member

sdepold commented Aug 30, 2010

Not sure if I understand your answer ^_^ You think it's OK to replace the readme stuff with a link to sequelizejs.com right? I would like that solution because this way I could focus more on the website.

@assertchris
Copy link
Author

assertchris commented Aug 30, 2010

Yeah - I think you should do it. I think you should also place your example scripts on the website - they have helped me a HUGE amount in figuring out how to use the lib...

@sdepold
Copy link
Member

sdepold commented Aug 30, 2010

Ah good idea :)

@sdepold
Copy link
Member

sdepold commented Aug 30, 2010

OK! Changed the readme :)

sdepold pushed a commit that referenced this issue Sep 15, 2022
* feat(oracle): add oracle dialect support (#1)

* feat(oracle): add oracle dialect support

* fix: addressing review comments (#7)

* fix: addressing review comments

* fix: minor fixes done (#9)

* fix: minor fixes to the review comments

* fix: merge from sequelize-v6

* fix: enable newly added unit tests for Oracle dialect

* fix: remove dangling comma (#13)

* fix: doc gen is fixed

* fix: autogenerate the primary constraint name (#14)

* fix: autogenerate the primary constraint name

* fix: remove trailing comma

* fix: make changes to ORADERBY as per v6 sync

* fix: move test-unit-oracle above test-unit-all

* fix: rename getInsertQueryReturnIntoBinds to
  populateInsertQueryReturnIntoBinds

* fix: reorder parameters for function populateInsertQueryReturnIntoBinds

* fix: incorporated review comments (#16)

* fix: incorporated review comments

* fix: modify string empty check with !
WikiRik pushed a commit that referenced this issue Sep 22, 2022
* feat(oracle): add oracle dialect support (#1)

* feat(oracle): add oracle dialect support

* fix: addressing review comments (#7)

* fix: addressing review comments

* fix: minor fixes done (#9)

* fix: minor fixes to the review comments

* fix: merge from sequelize-v6

* fix: enable newly added unit tests for Oracle dialect

* fix: remove dangling comma (#13)

* fix: doc gen is fixed

* fix: autogenerate the primary constraint name (#14)

* fix: autogenerate the primary constraint name

* fix: remove trailing comma

* fix: make changes to ORADERBY as per v6 sync

* fix: move test-unit-oracle above test-unit-all

* fix: rename getInsertQueryReturnIntoBinds to
  populateInsertQueryReturnIntoBinds

* fix: reorder parameters for function populateInsertQueryReturnIntoBinds

* fix: incorporated review comments (#16)

* fix: incorporated review comments

* fix: modify string empty check with !

* feat: support for Oracle DB 18c

* Oracle DB version change

* added stop-oracle for 18

* fix: changes to DB version query

* fix: cleanup

* fix: describetable query fix

* fix: dbVersion to remove round trip and 18.4 json test fix

* fix: removed dbversion

* fix: removed comment

* fix: testing a feature

* fix: testing a feature

* fix: testing a feature

* fix: testing a feature

* fix: testing a feature

* fix: testing a feature

* fix: test

* fix: test

* fix: test

* fix: test

* fix: using semver to coerce version sring

* fix: update to instant client latest version for oracle db 21c

* fix: update to oracledb version in package.json

* fix: update lockfile

* fix: remove duplicate privileges.sql and wait-until-healthy.sh

* fix: changes to start-oracle alias

* fix: changes to start-oracle alias

* fix: changes to start-oracle alias

Co-authored-by: Sudarshan Soma <48428602+sudarshan12s@users.noreply.github.com>
This issue was closed.
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

No branches or pull requests

2 participants