Skip to content

bugfix/response handlers return type#12

Merged
baywet merged 9 commits intomainfrom
bugfix/response-handlers-return-type
Feb 16, 2021
Merged

bugfix/response handlers return type#12
baywet merged 9 commits intomainfrom
bugfix/response-handlers-return-type

Conversation

@baywet
Copy link
Member

@baywet baywet commented Jan 7, 2021

  • fixes response handler return type for typescript
  • aligns switch style
  • fixes an issue where unresolved types would return empty
  • fixes java classes casing
  • fixes response handler type for java
  • adds missing imports for java - fixes imports duplication
  • updates CSharp debug location
  • fixes casing for default return type
  • adds default usings at the class level and not namespace

Note: there's still a bug in the CSHarp writer because of the recent change of not using namespaces as an entry point. The mapping needs to be updated like Java/Tyepescript, and the usings need to be tested
Note: There's still a bug in Java for the import for the GraphClient. It needs to do relative calculation like typescript does.

@baywet baywet requested a review from darrelmiller as a code owner January 7, 2021 17:52
@baywet baywet self-assigned this Jan 7, 2021
@baywet baywet merged commit 28e9aa8 into main Feb 16, 2021
@baywet baywet deleted the bugfix/response-handlers-return-type branch February 16, 2021 13:38
@baywet baywet added this to the MVP milestone Feb 16, 2021
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