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
-
I've realised that having the
level_id
in theusers
table is absolutely pointless. The level data is stored in theexperiences
table, and thegetLevel()
method is using that, so thatlevel_id
data in theusers
table 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