-
Notifications
You must be signed in to change notification settings - Fork 16
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
Conflict with Model Names & Different Model Identifiers #23
Comments
Looks like it's something that specifically started happening with M2+? Happens with M3s as well. 2024-09-23 17:44:59 - INFO - Could not find a model ID in Snipe-IT for: MacBook Pro (16-inch, Nov 2023) Model Name: MacBook Pro (16-inch, Nov 2023) Model Name: MacBook Pro (16-inch, Nov 2023) |
That validation bug was fixed in Snipe-IT already. If you update to latest on master Snipe-IT, can you let me know if you're still seeing this issue? |
Our Snipe Instance is on Snipe-IT version:v7.0.11 build 14904 (g6c0cf9447) but it's hosted for us So I'll need to take a look. It is fixed in 7.0.12? |
Yep - should be. Can you send an email to [email protected] with your hosted subdomain so we can get you updated to latest? |
I've noticed an issue where Model Names can cause conflicts where the Model Identifier differ and was failing the script out.
Model:MacBook Pro (16-inch, 2023)
Model Identifier:Mac14,10
M2 Pro
Model:MacBook Pro (16-inch, 2023)
Model Identifier:Mac14,6
M2 Max
I was getting the failure
2024-09-23 16:51:44 - INFO - Processing entry 3 out of 151 - Device Name: *** - Device ID: ****
2024-09-23 16:51:44 - INFO - Could not find a model ID in Snipe-IT for: MacBook Pro (16-inch, 2023)
Traceback (most recent call last):
File "", line 790, in
create_snipe_model(newmodel)
File "", line 558, in create_snipe_model
modelnumbers[jsonresponse['payload']['model_number']] = jsonresponse['payload']['id']
~~~~~~~~~~~~~~~~~~~~~~~^^^^^^
TypeError: 'NoneType' object is not subscriptable
Adding a new Model with the Model Identifier bypassed the error
The text was updated successfully, but these errors were encountered: