Skip to content

realign user API endpoints#11963

Open
matmair wants to merge 10 commits into
inventree:masterfrom
matmair:refactor-user-endpoints
Open

realign user API endpoints#11963
matmair wants to merge 10 commits into
inventree:masterfrom
matmair:refactor-user-endpoints

Conversation

@matmair
Copy link
Copy Markdown
Member

@matmair matmair commented May 18, 2026

realign user API endpoints to make it clearer which one are only applicable to the current user

Required Client Changes

@matmair matmair self-assigned this May 18, 2026
@matmair matmair added api Relates to the API refactor labels May 18, 2026
@netlify
Copy link
Copy Markdown

netlify Bot commented May 18, 2026

Deploy Preview for inventree-web-pui-preview ready!

Name Link
🔨 Latest commit 60ee236
🔍 Latest deploy log https://app.netlify.com/projects/inventree-web-pui-preview/deploys/6a0e2fe013c7460007e91715
😎 Deploy Preview https://deploy-preview-11963--inventree-web-pui-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 82 (🔴 down 9 from production)
Accessibility: 82 (no change from production)
Best Practices: 100 (no change from production)
SEO: 78 (no change from production)
PWA: -
View the detailed breakdown and full score reports
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@SchrodingersGat SchrodingersGat added breaking Indicates a major update or change which breaks compatibility User Interface Related to the frontend / User Interface labels May 18, 2026
@SchrodingersGat
Copy link
Copy Markdown
Member

@matmair I've created linked issues for the app and the python bindings - these will need to be updated too

@codecov
Copy link
Copy Markdown

codecov Bot commented May 19, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.44%. Comparing base (53d78be) to head (60ee236).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #11963      +/-   ##
==========================================
- Coverage   91.46%   91.44%   -0.02%     
==========================================
  Files         974      974              
  Lines       50704    50704              
==========================================
- Hits        46374    46366       -8     
- Misses       4330     4338       +8     
Flag Coverage Δ
backend 89.27% <100.00%> (-0.02%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Components Coverage Δ
Backend Apps 91.76% <ø> (-0.01%) ⬇️
Backend General 93.49% <ø> (ø)
Frontend ∅ <ø> (∅)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@matmair
Copy link
Copy Markdown
Member Author

matmair commented May 19, 2026

change for inventree-python is in; I will try my hand at the app but i have not done anything in dart properly in years

@matmair
Copy link
Copy Markdown
Member Author

matmair commented May 20, 2026

@SchrodingersGat the only thing blocking the CI is the failing test that is adresses in inventree/inventree-python#303

@matmair matmair marked this pull request as ready for review May 20, 2026 22:04
@matmair matmair requested a review from SchrodingersGat as a code owner May 20, 2026 22:04
@matmair matmair added this to the 1.4.0 milestone May 21, 2026
@SchrodingersGat
Copy link
Copy Markdown
Member

@matmair do you have an urgent need for this, or is an idealistic change? This will introduce a major breaking change for any clients (i know I'll have a few to manage) - can we make this a 2.0.0?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api Relates to the API breaking Indicates a major update or change which breaks compatibility refactor User Interface Related to the frontend / User Interface

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants