Skip to content

Conversation

@jhoward-lm
Copy link
Contributor

@jhoward-lm jhoward-lm commented Jun 27, 2024

Update to use protobom storage backend fixes

Description

This PR updates the protobom/storage dependency to v0.1.3 to address the following issues:

  • document type enum value fix
  • use transactional client for storing so changes are rolled back when error occurs

Type of change

  • Bug fix (non-breaking change which fixes an issue)

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules
  • I have checked my code and corrected any misspellings

Solves the following issues:

- document type enum value fix
- use transactional client for storing so changes are rolled back when error occurs

Signed-off-by: Jonathan Howard <jonathan.w.howard@lmco.com>
@jhoward-lm jhoward-lm added the bug Something isn't working label Jun 27, 2024
@jhoward-lm jhoward-lm self-assigned this Jun 27, 2024
@jhoward-lm jhoward-lm requested a review from a team as a code owner June 27, 2024 14:40
Signed-off-by: Jonathan Howard <jonathan.w.howard@lmco.com>
Copy link
Contributor

@mfrystacky mfrystacky left a comment

Choose a reason for hiding this comment

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

:shipit:

@jhoward-lm jhoward-lm merged commit 4e558e5 into main Jun 27, 2024
@jhoward-lm jhoward-lm deleted the ent-storage-backend-fixes branch June 27, 2024 15:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants