fix: proper aggregation of strategies

[gastonfournier]

About the changes

Open API is creating 2 resources under the same URL path, we want them aggregated:

$ curl -s https://us.app.unleash-hosted.com/ushosted/docs/openapi.json | jq '.paths."/api/admin/strategies/{name}" | keys'
[
  "delete",
  "get"
]
gaston@gaston-Summit-E16Flip-A12UCT: ~/poc/terraform-provider-unleash on main [!$]
$ curl -s https://us.app.unleash-hosted.com/ushosted/docs/openapi.json | jq '.paths."/api/admin/strategies/{strategyName}" | keys'
[
  "put"
]

Note one is under {name} while the other is under {strategyName}

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
unleash-monorepo-frontend	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Aug 9, 2023 1:11pm

1 Ignored Deployment

Name	Status	Preview	Comments	Updated (UTC)
unleash-docs	⬜️ Ignored (Inspect)	Visit Preview		Aug 9, 2023 1:11pm

[thomasheartman]

The name/strategyName fix is very welcome! 💯 I wasn't aware that booleans couldn't use enums. Do you have the docs for that anywhere? I'd ideally like to keep that as is because it accurately describes that state of the response, but if it causes issues, then changing it isn't really a big deal 😄

[gastonfournier]

The name/strategyName fix is very welcome! 100 I wasn't aware that booleans couldn't use enums. Do you have the docs for that anywhere? I'd ideally like to keep that as is because it accurately describes that state of the response, but if it causes issues, then changing it isn't really a big deal smile

I confirmed that the boolean is fine for openapi spec 3.0 (we're on 3.0.3). Version 3.1 introduces const which would be more meaningful in this case, but I'm keeping it as is for now