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

Using existing User model #1132

Closed
3 tasks done
dgillier opened this issue Sep 10, 2022 · 3 comments
Closed
3 tasks done

Using existing User model #1132

dgillier opened this issue Sep 10, 2022 · 3 comments

Comments

@dgillier
Copy link

Is your feature request related to a problem? Please describe.

Unless you are creating just a blog, most other use are about adding a blog to an existing app.
And in this situation, there is already a User model and duplicating it make things more difficult...

Describe the solution you'd like

I looked at your User table and there is just a few more fields but nothing used intensively...

I suggest to add some metadata to the existing User model by using, for example, a package like https://github.com/99codes/laravel-meta-attributes...

Describe alternatives you've considered

No response

Additional context

No response

Validations

@PaulChase
Copy link

I totally agree.

The new canvas users table is not necessary, it could have just been merged with the existing users table.

@austintoddj
Copy link
Owner

It used to be that way. There's a discussion and a related issue that dive a little deeper into it.

@austintoddj
Copy link
Owner

Closing this because I believe a separation is applicable to a number of situations. While it isn't necessary in all cases, I'd like to keep it as is for the time being. Thanks @dgillier for offering the suggestion though!

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

3 participants