Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Comments

Close side panel

Generated v1.0 models and request builders using Typewriter#814

Closed
github-actions[bot] wants to merge 3 commits intodevmicrosoftgraph/msgraph-sdk-java:devfrom
v1.0/pipelinebuild/51834microsoftgraph/msgraph-sdk-java:v1.0/pipelinebuild/51834Copy head branch name to clipboard
Closed

Generated v1.0 models and request builders using Typewriter#814
github-actions[bot] wants to merge 3 commits intodevmicrosoftgraph/msgraph-sdk-java:devfrom
v1.0/pipelinebuild/51834microsoftgraph/msgraph-sdk-java:v1.0/pipelinebuild/51834Copy head branch name to clipboard

Conversation

@github-actions
Copy link
Contributor

This pull request was automatically created by the GitHub Action, create pull request.

The commit hash is 586ed44.

Important Check for unexpected deletions or changes in this PR.

Copy link
Member

@baywet baywet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

3-9-0_to_java.zip

@MIchaelMainer the fact that some teams entities changed their properties types from identityset to chatmessageidentityset (which inherits from the former) is potentially a breaking change for people creating/updating information.

See this example

Copy link
Member

@baywet baywet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

blocking for now as discussed offline

@MIchaelMainer
Copy link
Collaborator

@baywet I think we should treat this as a blocking for all of the SDKs. We should close out all PRs with these changes.

microsoftgraph/msgraph-metadata@d6fa7f7

@baywet
Copy link
Member

baywet commented Jun 23, 2021

agreed, waiting on @darrelmiller to confirm after he talks with the owner.

@baywet
Copy link
Member

baywet commented Jun 29, 2021

Closing as replaced by #815

@baywet baywet closed this Jun 29, 2021
@baywet baywet deleted the v1.0/pipelinebuild/51834 branch June 29, 2021 10:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.