Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

D42 14600 - Lenovo Warranty Check Error #31

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

osanchez42
Copy link
Contributor

Updates to script for the new lenovo ibm support API. do not merge until ticket is marked as resolved

@osanchez42 osanchez42 requested a review from cscaglioned42 May 19, 2020 23:10
@osanchez42 osanchez42 self-assigned this May 19, 2020
# Warranties
if 'Warranty' in item and len(item['Warranty']) > 0:
warranties = item['Warranty']
else:
continue

data.clear()
serial = item['ID']
serial = item['Serial']
Copy link
Contributor

Choose a reason for hiding this comment

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

It doesn't look like this was even being used.

Copy link
Contributor Author

@osanchez42 osanchez42 May 20, 2020

Choose a reason for hiding this comment

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

Yea, since we were waiting to get the script validated the changes to the script for the new API were based solely on the API docs. The docs stated that the following items would be returned. Serial number was not mentioned so I assumed serial number was listed as ID.

InWarranty - Whether this product is in warranty
Purchased - Date this product was purchased Will not be available if we do not have the purchase information (product was not registered)
Shipped - Date this product was shipped
Warranty - List of warranties
ID
Type
Name
Start
End
Contract - List of contract warranties. Available only if the product is part of a contract
Contract
Quantity
ItemNumber
ChargeCode
SLA
EntitlementCode
Status
Start
End

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