Skip to content

OBPIH-7193: add product inventory snapshot transaction type#5238

Merged
awalkowiak merged 2 commits intodevelopfrom
ft/OBPIH-7193-inventory-snapshot-transaction-type
May 15, 2025
Merged

OBPIH-7193: add product inventory snapshot transaction type#5238
awalkowiak merged 2 commits intodevelopfrom
ft/OBPIH-7193-inventory-snapshot-transaction-type

Conversation

@ewaterman
Copy link
Member

✨ Description of Change

Link to GitHub issue or Jira ticket: https://pihemr.atlassian.net/browse/OBPIH-7193

Description: Add the new "Product Inventory Snapshot" transaction type and switch cycle count to use it.


📷 Screenshots & Recordings (optional)

Cycle counts can successfully create the new "product inventory snapshot" transactions after the count is submitted:

Screenshot from 2025-04-29 11-54-19

And last counted date is still updated correctly:

Screenshot from 2025-04-29 11-53-48

@ewaterman ewaterman self-assigned this Apr 29, 2025
@github-actions github-actions bot added type: feature A new piece of functionality for the app domain: backend Changes or discussions relating to the backend server flag: schema change Hilights a pull request that contains a change to the database schema labels Apr 29, 2025
@codecov
Copy link

codecov bot commented Apr 29, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 8.27%. Comparing base (e6c82a5) to head (e428b1f).
⚠️ Report is 150 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff              @@
##             develop   #5238      +/-   ##
============================================
- Coverage       8.31%   8.27%   -0.04%     
- Complexity       971     982      +11     
============================================
  Files            638     641       +3     
  Lines          43246   43288      +42     
  Branches       10511   10520       +9     
============================================
- Hits            3596    3584      -12     
- Misses         39097   39154      +57     
+ Partials         553     550       -3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

<insert tableName="transaction_type">
<column name="id" value="12" />
<column name="version" value="0" />
<column name="name" value="Product Inventory Snapshot" />
Copy link
Member Author

@ewaterman ewaterman May 6, 2025

Choose a reason for hiding this comment

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

@awalkowiak I realized later that you wanted the name to be "Product Snapshot", so I can change it if you prefer

Copy link
Collaborator

Choose a reason for hiding this comment

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

@ewaterman, actually, I wanted "Inventory Snapshot". However looks like there is still a debate between @jmiranda and Kelsey on how it should be named (see this comment https://pihemr.atlassian.net/browse/OBPIH-7193?focusedCommentId=163366), so let's confirm today which is the way to go.

<insert tableName="transaction_type">
<column name="id" value="12" />
<column name="version" value="0" />
<column name="name" value="Product Inventory Snapshot" />
Copy link
Collaborator

Choose a reason for hiding this comment

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

@ewaterman, actually, I wanted "Inventory Snapshot". However looks like there is still a debate between @jmiranda and Kelsey on how it should be named (see this comment https://pihemr.atlassian.net/browse/OBPIH-7193?focusedCommentId=163366), so let's confirm today which is the way to go.

@ewaterman ewaterman requested a review from awalkowiak May 14, 2025 18:08
@awalkowiak awalkowiak merged commit 352f7a1 into develop May 15, 2025
9 checks passed
@awalkowiak awalkowiak deleted the ft/OBPIH-7193-inventory-snapshot-transaction-type branch May 15, 2025 08:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

domain: backend Changes or discussions relating to the backend server flag: schema change Hilights a pull request that contains a change to the database schema type: feature A new piece of functionality for the app

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants