-
Notifications
You must be signed in to change notification settings - Fork 73
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
Make independent mapping rules always enabled when APIAP is on #1133
Make independent mapping rules always enabled when APIAP is on #1133
Conversation
c588425
to
09fa0ef
Compare
app/views/api/integrations/apicast/shared/_mapping_rules.html.slim
Outdated
Show resolved
Hide resolved
app/views/shared/provider/navigation/service/_integration.html.slim
Outdated
Show resolved
Hide resolved
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.
As said in one of my comment no need to keep the rolling update as it will just be part of API as product. Nobody is using it anyway in SaaS.
7405cc6
to
c7201fd
Compare
c7201fd
to
9b18548
Compare
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.
LGTM 👍 But wait for 1 more review please 😉
@@ -44,7 +44,7 @@ def destroy | |||
private | |||
|
|||
def authorize! | |||
provider_can_use!(:independent_mapping_rules) | |||
provider_can_use!(:api_as_product) |
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.
It should be provider_can_use!(:api_as_product) || provider_can_use!(:independent_mapping_rules)
@thomasmaas ?
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.
I would say provider_can_use!(:api_as_product)
is enough, unless we have customers on saas who are using this? As in our next on prem release all providers can and will use api_as_product
THREESCALE-3341 We need to make independent_mapping_rules always enabled for services acting as product.
9b18548
to
2cb24b8
Compare
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.
OK lets just leave the RU like that for now, and open the option to have it in SaaS (though not good UX)
What this PR does / why we need it:
We need to make independent_mapping_rules always enabled for
services acting as product.
Which issue(s) this PR fixes
THREESCALE-3341
Depends on #1156