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

accountsBoost overwrites user-provided options #607

Closed
moltar opened this issue Mar 6, 2019 · 7 comments · Fixed by #1258
Closed

accountsBoost overwrites user-provided options #607

moltar opened this issue Mar 6, 2019 · 7 comments · Fixed by #1258

Comments

@moltar
Copy link

moltar commented Mar 6, 2019

If I supply my own db via accountsBoost, it will always get overwritten:

options.db = new DatabaseManager({

This basically locks accountsBoost only to mongo implementation, and will block @accounts/typeorm, which is very desirable!

@pradel
Copy link
Member

pradel commented Mar 6, 2019

You are right this part should be updated to also support typeorm.
Pull request welcome 🙌

@agustif
Copy link
Contributor

agustif commented Apr 5, 2019

You are right this part should be updated to also support typeorm.
Pull request welcome 🙌

I would love to try to get this done, but would also highly appreciate any guidance on where should I start.

I myself am planning to use AccountsJS and preferably with boost with a TypeORM/PostgreSQL project, and was planning to copy the implementation from @brikir prime-core which I got to know about AccountsJS to start from...

Would also love to add PostgreSQL to the docs will I am at it...

@agustif
Copy link
Contributor

agustif commented Apr 14, 2019

Well. I don't know about accountsBoost, but I'm gonna go ahead and try to modify one of the examples to use postgresql instead of the default mongo and try to document submit the example as a PR back once it's finished.

Im going to basically follow onto the lines of @birkir on his implementation of this on primecms, which I recently saw made him develop accounts-js/typeorm package which should allow to do this outside of accountsBoost pretty easy than from start!

Hope to be able to do it by myself, will report back soon!

@agustif
Copy link
Contributor

agustif commented Apr 15, 2019

I'm a little bit stuck, since I don't know how to create the postgres database (I do create the postgres service with the docker file, might share my progress so far later today either if I succeed or not

Update:

OMG I don't even believe I got it working by myself, It really needs a lot of polish, and @birkir really deserves all the credit for creating the @accounts/typeorm package.

But it's working!

What would be the best way to commit this back to the community, It really needs some work to become a mergeable PR, but maybe I can just send it as a draft PR on a new branch called examples/graphql-server-typeorm-postgres and recieve some feedback as comments.

Btw I'm also trying to create a new UI version in antd, and would like to end up adding NextJS to the mix, might commit that later down the road!

I'm just so happy I could get it working!! 😄

@birkir
Copy link
Contributor

birkir commented Apr 15, 2019

Good to hear @agustif

You can start by sharing a PR or branch with your changes. We can give you a hand if necessary or feedback.

UI would be great!

@agustif
Copy link
Contributor

agustif commented Apr 15, 2019

Good to hear @agustif

You can start by sharing a PR or branch with your changes. We can give you a hand if necessary or feedback.

UI would be great!

OK, cleaning a bit opening a PR soon!

@agustif
Copy link
Contributor

agustif commented Apr 20, 2019

Hey @birkir I did start it, but it's really rough on the edges.

#654

I basically copied the implementation from prime-core and adapted it to fit the previous example.

Do you have any feedback on how to proceed? I do really don't know what happened with removing the package lock making tests fail.

Maybe I need to split the two examples in two PR/branches to make this easier to evaluate?

Thanks for helping out, should I open a new Issue about Examples TypeORM/Postgres and leave this issue alone since this is just somewhat orthogonal to the accountsBoost issue.

Thanks again and sorry for the mess!

@darkbasic darkbasic added the 1.0 label Nov 22, 2023
@darkbasic darkbasic added this to the 1.0 milestone Nov 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants