Skip to content

Add error message for missing TLD - #12789

Merged
SchrodingersGat merged 3 commits into
inventree:masterfrom
SchrodingersGat:tld-error
Sep 6, 2026
Merged

Add error message for missing TLD#12789
SchrodingersGat merged 3 commits into
inventree:masterfrom
SchrodingersGat:tld-error

Conversation

@SchrodingersGat

Copy link
Copy Markdown
Member

Add specific error message to help sysadmin setup

@SchrodingersGat SchrodingersGat added the setup Relates to the InvenTree setup / installation process label Sep 5, 2026
@netlify

netlify Bot commented Sep 5, 2026

Copy link
Copy Markdown

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

Name Link
🔨 Latest commit 7f0c761
🔍 Latest deploy log https://app.netlify.com/projects/inventree-web-pui-preview/deploys/6a9cbb35882a4f00082429ae
😎 Deploy Preview https://deploy-preview-12789--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: 92 (🔴 down 2 from production)
Accessibility: 82 (no change from production)
Best Practices: 92 (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 documentation backport Apply this label to a PR to enable auto-backport action backport-to-1.5.x labels Sep 5, 2026
@codecov

codecov Bot commented Sep 5, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 22.22222% with 14 lines in your changes missing coverage. Please review.
✅ Project coverage is 87.13%. Comparing base (5b4c813) to head (7f0c761).

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #12789      +/-   ##
==========================================
- Coverage   87.14%   87.13%   -0.02%     
==========================================
  Files        1483     1483              
  Lines      100725   100724       -1     
  Branches    11524    11515       -9     
==========================================
- Hits        87781    87765      -16     
- Misses      12882    12895      +13     
- Partials       62       64       +2     
Flag Coverage Δ
backend 91.17% <22.22%> (-0.03%) ⬇️

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

Components Coverage Δ
Backend Apps 92.34% <ø> (-0.11%) ⬇️
Backend General 93.98% <ø> (-0.73%) ⬇️
Frontend 79.86% <ø> (+0.12%) ⬆️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@matmair matmair left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Lgtm; should we add an error code for this?

@matmair matmair added this to the 1.6.0 milestone Sep 5, 2026
@SchrodingersGat

Copy link
Copy Markdown
Member Author

@matmair nice idea, I have added an error code for this

@SchrodingersGat
SchrodingersGat merged commit 321489a into inventree:master Sep 6, 2026
56 of 57 checks passed
@SchrodingersGat
SchrodingersGat deleted the tld-error branch September 6, 2026 05:00
@github-actions

github-actions Bot commented Sep 6, 2026

Copy link
Copy Markdown
Contributor

💚 All backports created successfully

Status Branch Result
1.5.x

Questions ?

Please refer to the Backport tool documentation and see the Github Action logs for details

SchrodingersGat added a commit that referenced this pull request Sep 6, 2026
* Add error message for missing TLD

* add error code for missing TLD

(cherry picked from commit 321489a)

Co-authored-by: Oliver <oliver.henry.walters@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport Apply this label to a PR to enable auto-backport action backport-to-1.5.x documentation setup Relates to the InvenTree setup / installation process

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants