Skip to content

Object.getOwnPropertyDescriptor() can return 'undefined' if property descriptor not found.#18148

Merged
mhegazy merged 1 commit into
microsoft:masterfrom
avchugaev:master
Sep 7, 2017
Merged

Object.getOwnPropertyDescriptor() can return 'undefined' if property descriptor not found.#18148
mhegazy merged 1 commit into
microsoft:masterfrom
avchugaev:master

Conversation

@avchugaev

@avchugaev avchugaev commented Aug 30, 2017

Copy link
Copy Markdown
Contributor

Object.getOwnPropertyDescriptor() can return 'undefined' if property descriptor not found.
See https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Object/getOwnPropertyDescriptor

@msftclas

Copy link
Copy Markdown

@achugaev93,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla.microsoft.com.

It will cover your contributions to all Microsoft-managed open source projects.
Thanks,
Microsoft Pull Request Bot

@avchugaev avchugaev changed the title Object.getOwnPropertyDescriptor() returns 'undefined' if property descriptor not found. Object.getOwnPropertyDescriptor() can return 'undefined' if property descriptor not found. Aug 30, 2017
@msftclas

Copy link
Copy Markdown

@achugaev93, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request.

Thanks, Microsoft Pull Request Bot

@avchugaev

Copy link
Copy Markdown
Contributor Author

Can you please review this changes?

@mhegazy mhegazy merged commit de313ff into microsoft:master Sep 7, 2017
@mhegazy

mhegazy commented Sep 7, 2017

Copy link
Copy Markdown
Contributor

Thanks for your PR. For future references, please file an issue first to track changes.

@avchugaev

Copy link
Copy Markdown
Contributor Author

@mhegazy Ok, thanks for you advice.

@microsoft microsoft locked and limited conversation to collaborators Jun 14, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants