Skip to content

Conversation

@derricklam-zendesk
Copy link

Summary

The AppConfig field in CreateGroupPushMappingRequest should be a pointer to properly support that this field is optional:

appConfig - Additional app configuration for group push mappings. Currently only required for Active Directory.

Currently when CreateGroupPushMappingRequest is initialized, the SDK will serialize the appConfig field with {"type":""}.

This causes the following error when trying to call CreateGroupPushMapping :

(status: 400, body: {"errorCode":"E0000003","errorSummary":"The request body was not well-formed.","errorCauses":[]}): 400 Bad Request

Fixes #

Type of PR

  • Bug Fix (non-breaking fixes to existing functionality)
  • New Feature (non-breaking changes that add new functionality)
  • Documentation update
  • Test Updates
  • Other (Please describe the type)

Test Information

  • My PR required test updates

Go Version:
Os Version:
OpenAPI Spec Version:

Signoff

  • I have submitted a CLA for this PR
  • Each commit message explains what the commit does
  • I have updated documentation to explain what my PR does
  • My code is covered by tests if required
  • I ran make fmt on my code
  • I did not edit any automatically generated files

@derricklam-zendesk
Copy link
Author

cc @pranav-okta

@pranav-okta
Copy link
Contributor

Hey @derricklam-zendesk, thanks for the contribution!

This is auto-generated code created by the openapi-generator-cli using the spec file located in the .generator directory. You might want to change either the okta-management-APIs-oasv3-noEnums-inheritance.yaml or the model_simple.mustache

@derricklam-zendesk
Copy link
Author

Hey @derricklam-zendesk, thanks for the contribution!

This is auto-generated code created by the openapi-generator-cli using the spec file located in the .generator directory. You might want to change either the okta-management-APIs-oasv3-noEnums-inheritance.yaml or the model_simple.mustache

@pranav-okta thanks I see. Can I get your assistance with generating this file? When I ran make generate it modified over 3k files

@pranav-okta
Copy link
Contributor

Can you add the code that you're testing with?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants