Skip to content

Update signing period buttons, binary files, and ports#11

Merged
raymondjstone merged 1 commit into
masterfrom
develop
Apr 9, 2026
Merged

Update signing period buttons, binary files, and ports#11
raymondjstone merged 1 commit into
masterfrom
develop

Conversation

@raymondjstone
Copy link
Copy Markdown
Owner

  • Reordered "Current Signing Period" and "Previous Signing Period" buttons in JsaReport.razor for improved logic and UI consistency.
  • Updated LFS pointers for JobTracker-x64/x86 zip parts to new versions.
  • Changed application URLs in launchSettings.json to use ports 8146/8147 instead of 7046/7047.

- Reordered "Current Signing Period" and "Previous Signing Period" buttons in JsaReport.razor for improved logic and UI consistency.
- Updated LFS pointers for JobTracker-x64/x86 zip parts to new versions.
- Changed application URLs in launchSettings.json to use ports 8146/8147 instead of 7046/7047.
Copilot AI review requested due to automatic review settings April 9, 2026 10:21
@raymondjstone raymondjstone merged commit 611204d into master Apr 9, 2026
4 checks passed
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 9, 2026

Qodana for JVM

It seems all right 👌

No new problems were found according to the checks applied

☁️ View the detailed Qodana report

Contact Qodana team

Contact us at qodana-support@jetbrains.com

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates local development settings and UI ordering while bumping bundled portable JobTracker artifacts tracked via Git LFS.

Changes:

  • Reordered “Previous Signing Period” and “Current Signing Period” buttons in JsaReport.razor.
  • Updated Git LFS pointers for Portable/JobTracker-x64/x86 split zip parts.
  • Changed launchSettings.json local URLs to ports 8146/8147.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
Properties/launchSettings.json Updates local dev application URLs/ports.
Components/Pages/JsaReport.razor Reorders signing-period quick range buttons.
Portable/JobTracker-x86.zip.001 Updates LFS pointer for x86 zip part.
Portable/JobTracker-x86.zip.002 Updates LFS pointer for x86 zip part.
Portable/JobTracker-x64.zip.001 Updates LFS pointer for x64 zip part.
Portable/JobTracker-x64.zip.002 Updates LFS pointer for x64 zip part.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

"dotnetRunMessages": true,
"launchBrowser": true,
"applicationUrl": "https://localhost:7046;http://localhost:7046",
"applicationUrl": "https://localhost:8147;http://localhost:8147",
Copy link

Copilot AI Apr 9, 2026

Choose a reason for hiding this comment

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

In the https profile, applicationUrl specifies both https://localhost:8147 and http://localhost:8147. Binding HTTP and HTTPS to the same port will typically fail at runtime (port conflict) and also doesn’t match the PR description of using ports 8146/8147. Use distinct ports (e.g., http://localhost:8146 + https://localhost:8147) or remove the HTTP URL from the https profile.

Suggested change
"applicationUrl": "https://localhost:8147;http://localhost:8147",
"applicationUrl": "https://localhost:8147;http://localhost:8146",

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants