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

Ensure all resources are correctly added to the list resources #299

Merged
merged 2 commits into from
Oct 16, 2024

Conversation

Wambere
Copy link
Contributor

@Wambere Wambere commented Oct 16, 2024

IMPORTANT: Where possible all PRs must be linked to a Github issue

Fixes #289

Engineer Checklist

  • I have run ./gradlew spotlessApply to check my code follows the project's style guide
  • I have built and run the efsity jar to verify my change fixes the issue and/or does not break the application

@@ -256,12 +256,14 @@ def main(
del _["code"]
issues.append(_)
logging.error(json_response)
logging.info(inventory_creation_response.text)
logging.info("GROUPS: " + str(groups_created))
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Adding these to help with debugging. If you are running an import and the Groups or Lists are empty, then there is obviously an issue somewhere

@pld pld enabled auto-merge (squash) October 16, 2024 19:34
@pld pld merged commit 008a34a into main Oct 16, 2024
4 checks passed
@pld pld deleted the 289-fix-list branch October 16, 2024 19:39
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.

[Importer] Adding the Group Resources representing Inventories to the Inventory List entries
2 participants