Removing level_id from Users' Model
#65
Closed
cjmellor
announced in
Announcements
Replies: 1 comment
-
|
I agree, if it doesn't affect anything, is it safe to remove the migration? If so, which one? Cheers 🎉 EDIT: Remove the one that connects the relation to the users table ( |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
I've realised that having the
level_idin theuserstable is absolutely pointless. The level data is stored in theexperiencestable, and thegetLevel()method is using that, so thatlevel_iddata in theuserstable is just extra fluff that isn't needed.I aren't sure if this would be classed as a breaking change 🤔
Beta Was this translation helpful? Give feedback.
All reactions