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
Move product schema changes out of BMO extension #462
Conversation
d0b3d7a
to
3125b2e
Compare
3125b2e
to
8ba3f38
Compare
Normally, this is done by Bugzilla::ModPerl. Do the same for other configurations.
…l in the "disable my account" description
@@ -84,6 +84,32 @@ sub get_param_list { | |||
name => 'strict_isolation', | |||
type => 'b', | |||
default => 0 | |||
}, | |||
{ | |||
name => 'always_filleable_groups', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry for the drive-by review, was just having a quick skim. Is filleable
spelled right there? Seems like that might cause problems down the line?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably should have a better name, yeah.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey, @mohawk2 if you would be so kind to take a look at bugzilla/harmony#25?
Some stuff is happening in that repo because today someone submitted like 17 PRs \o/
("harmony" is "bugzilla 6 should be based on bmo" project.)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
be484f7
to
ec2404b
Compare
21de5c1
to
bc66a21
Compare
This looks like it wants a rebase, as there are conflicts? Do you need further reviewing? |
We're doing this in the upstream repo, bugzilla/harmony#31. |
No description provided.