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

Fixes for test result updates #6514

Merged
merged 11 commits into from
Feb 19, 2024

Conversation

SchrodingersGat
Copy link
Member

@SchrodingersGat SchrodingersGat commented Feb 19, 2024

Improvements and fixes following on from #6430

  • Fix UI issues
  • Improve logic for data migration and API
  • Add data migration to catch potential edge case missed in previous PR

- Can be passed undefined value in some cases
- Add in "pass test" button
- Only look at ancestor parts *above* the existing part
- Only look above!
@SchrodingersGat SchrodingersGat added the bug Identifies a bug which needs to be addressed label Feb 19, 2024
@SchrodingersGat SchrodingersGat added this to the 0.14.0 milestone Feb 19, 2024
Copy link

netlify bot commented Feb 19, 2024

Deploy Preview for inventree-web-pui-preview canceled.

Name Link
🔨 Latest commit fef7fc8
🔍 Latest deploy log https://app.netlify.com/sites/inventree-web-pui-preview/deploys/65d2db4f83bfb50008b4c1dc

- Fixes (probably rare) edge case in previous data migration
- Embolden required test templates
- Add filter for "has_results" on the PartTestTemplate API endpoint
@SchrodingersGat SchrodingersGat added migration Data or schema migrations api Relates to the API labels Feb 19, 2024
@SchrodingersGat SchrodingersGat merged commit a74b29f into inventree:master Feb 19, 2024
26 checks passed
@SchrodingersGat SchrodingersGat deleted the test-results branch February 19, 2024 05:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api Relates to the API bug Identifies a bug which needs to be addressed migration Data or schema migrations
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant