Skip to content
This repository has been archived by the owner on Oct 3, 2024. It is now read-only.

Add rotera item api #172

Merged
merged 5 commits into from
May 4, 2023
Merged

Add rotera item api #172

merged 5 commits into from
May 4, 2023

Conversation

letmaik
Copy link
Contributor

@letmaik letmaik commented May 1, 2023

Example: https://www.ikea.com/global/assets/rotera/resources/30221043.json

This allows to fetch 3D models from the returned URLs.

@vrslev
Copy link
Owner

vrslev commented May 2, 2023

Hi there! Everything seems good, but you need to add tests and run lints using pre-commit.

@vrslev
Copy link
Owner

vrslev commented May 2, 2023

Oh, and just ignore the typing check failing—it is pydantic issue

Signed-off-by: Maik Riechert <letmaik@outlook.com>
Signed-off-by: Maik Riechert <letmaik@outlook.com>
@vrslev
Copy link
Owner

vrslev commented May 3, 2023

Coverage still fails, please cover 28–29 lines

@letmaik
Copy link
Contributor Author

letmaik commented May 3, 2023

Coverage still fails, please cover 28–29 lines

Fixed.

@vrslev vrslev merged commit a979b59 into vrslev:main May 4, 2023
@letmaik letmaik deleted the rotera branch May 4, 2023 07:16
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.

2 participants