Skip to content

Release 2.1.0 - #331

Closed
Nikitha Chettiar (nikithauc) wants to merge 20 commits into
masterfrom
nikithauc/Release-2.1.0
Closed

Release 2.1.0#331
Nikitha Chettiar (nikithauc) wants to merge 20 commits into
masterfrom
nikithauc/Release-2.1.0

Conversation

@nikithauc

Copy link
Copy Markdown
Contributor

Bumping the version to 2.1.0 for changes in 2.1.0-Preview.1

Comment thread src/GraphRequestUtil.ts Outdated

export const serializeContent = (content: any): any => {
const className: string = content.constructor.name;
const className: string = content === undefined ? undefined : content.constructor.name;

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
const className: string = content === undefined ? undefined : content.constructor.name;
const className: string = content && content.constructor && content.constructor.name;

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done.

Comment thread src/HTTPClient.ts Outdated
Comment thread src/HTTPClient.ts
Comment thread src/HTTPClient.ts
Comment thread src/HTTPClient.ts
Comment thread src/Version.ts
@baywet

Vincent Biret (baywet) commented Oct 1, 2020

Copy link
Copy Markdown
Member

Nikitha Chettiar (@nikithauc) can I assume this one is replaced by #333 ?

@nikithauc

Copy link
Copy Markdown
Contributor Author

Vincent Biret (@baywet) #333 is not a replacement. There will another PR for release2.1.0

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.

5 participants