Skip to content

Conversation

@SHANDCRUZ
Copy link
Contributor

@SHANDCRUZ SHANDCRUZ commented Feb 3, 2021

Description

changed the python class to new style

Issues Resolved

None

Check List

  • New functionality includes testing.
    • All tests pass for Python 2.7+ & 3.4+($ tox).
  • New functionality has been documented in the README if applicable.
    • New functionality has been thoroughly documented in the examples (please include helpful comments).
    • New endpoints supported are updated in the endpoints-support.md file.
  • Changes are documented in the CHANGELOG file.

Copy link
Contributor

@VenkateshRavula VenkateshRavula left a comment

Choose a reason for hiding this comment

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

Update changelog and endpoint-support files with this resource. Also modify oneview_client.py file and test_oneview_client.py files respectively.

Refer the below PR - #28

Copy link
Contributor

@VenkateshRavula VenkateshRavula left a comment

Choose a reason for hiding this comment

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

Please modify endpoints file

@VenkateshRavula VenkateshRavula self-requested a review February 10, 2021 12:30
|<sub>/rest/appliance/trap-destinations/{id}</sub> |PUT | :white_check_mark: | :white_check_mark: | :white_check_mark: | :white_check_mark: | :white_check_mark: | :white_check_mark: | :white_check_mark: | :white_check_mark: |
|<sub>/rest/appliance/trap-destinations/{id}</sub> |DELETE | :white_check_mark: | :white_check_mark: | :white_check_mark: | :white_check_mark: | :white_check_mark: | :white_check_mark: | :white_check_mark: | :white_check_mark: |
| **Appliance Time and Locale Configuration**
|<sub>/rest/appliance/configuration/time-locale</sub> |GET | :white_check_mark: | :white_check_mark: | :white_check_mark: | :white_check_mark: | :white_check_mark: | :white_check_mark: | :white_check_mark: | :white_check_mark: |
Copy link
Contributor

Choose a reason for hiding this comment

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

keep white_check_mark only for api 2400

@SHANDCRUZ SHANDCRUZ merged commit 4c3ac0c into master Feb 19, 2021
@SHANDCRUZ SHANDCRUZ deleted the modified_py_code branch February 19, 2021 05:07
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.

4 participants