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

Use a different model for tokens #15

Closed
atymic opened this issue Mar 29, 2020 · 9 comments
Closed

Use a different model for tokens #15

atymic opened this issue Mar 29, 2020 · 9 comments
Labels
enhancement New feature or request

Comments

@atymic
Copy link

atymic commented Mar 29, 2020

For example, save auth against a Company model rather than a user, allowing multiple users to access the same connection.

Will fork and PR.

@jimmypuckett jimmypuckett added the enhancement New feature or request label Apr 3, 2020
@Dozorov
Copy link

Dozorov commented Aug 21, 2020

I need it too...

@jimmypuckett
Copy link
Member

@atymic Did you get anywhere on that PR?

@ViktorIvanisenko
Copy link

I took your module as an example of functionality, I want to implement the connection of multiple users in turn with different accounts.

@atymic
Copy link
Author

atymic commented Oct 8, 2020

I ended up just taking the quickbooks token class and not using the package.
I can PR if you'd like, but it would be a breaking release since it's very different to how it work rn.

@ViktorIvanisenko
Copy link

I ended up just taking the quickbooks token class and not using the package.
I can PR if you'd like, but it would be a breaking release since it's very different to how it work rn.

I also redesigned the token class and the client class. I take access to QuickBooks from the company tables.

@atymic
Copy link
Author

atymic commented Oct 10, 2020

Yep, that's how I did it.

@ViktorIvanisenko
Copy link

Yep, that's how I did it.

When I'm done, I'll show you the code.

@hyphen81
Copy link

hyphen81 commented Aug 7, 2021

@ViktorIvanisenko Would love to see an example of how you did this. I think I need this approach too.

@ssfinney
Copy link
Member

ssfinney commented Sep 10, 2021

This sounds like a great potential new feature. Example code is welcome.

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

No branches or pull requests

6 participants