Keenan Brock
Keenan Brock
@Fryguy Is there a timeframe that is best to merge these schema PRs?
rails 7.0 supports ruby 3.0 and 3.1 - so at least according to that, it should work
sigh. ping if you need to pair on this. no ideas at this time
@jrafanie so you are converting from relations to ids so we can avoid defining relations all together? (e.g.: `ext_management_system => ems` to `ems_id => ems.id`) Seems those are very non-rails...
1. Think `allow_null => false` is good here. 2. The allow null updates all the columns to the default value. 3. But since the update is part of a schema...
@jrafanie I see you closed https://github.com/ManageIQ/manageiq/pull/22923 Do we still want to go forward with this? Personally I really like having these defaults in the database, thought I know we've been...
@Guddetisandeep Do we still need these PRs now that you are using collection class?
yea, I'm still :+1: on this one
@Fryguy @agrare Do we want to cache the OS value (since it is basically static?) This has been hanging out for 4 years. would be nice to give a yea/nay...