Skip to content
Navigation Menu
Sign in
Appearance settings
Platform
AI CODE CREATION
GitHub Copilot
Write better code with AI
GitHub Copilot app
Direct agents from issue to merge
MCP Registry
Integrate external tools
DEVELOPER WORKFLOWS
Actions
Automate any workflow
Codespaces
Instant dev environments
Issues
Plan and track work
Code Review
Manage code changes
Code Quality
Enforce quality at merge
APPLICATION SECURITY
GitHub Advanced Security
Find and fix vulnerabilities
Code security
Secure your code as you build
Secret protection
Stop leaks before they start
EXPLORE
Why GitHub
Documentation
Blog
Changelog
Marketplace
View all features
Solutions
BY COMPANY SIZE
Enterprises
Small and medium teams
Startups
Nonprofits
BY USE CASE
App Modernization
DevSecOps
DevOps
CI/CD
View all use cases
BY INDUSTRY
Healthcare
Financial services
Manufacturing
Government
View all industries
View all solutions
Resources
EXPLORE BY TOPIC
AI
Software Development
DevOps
Security
View all topics
EXPLORE BY TYPE
Customer stories
Events & webinars
Ebooks & reports
Business insights
GitHub Skills
SUPPORT & SERVICES
Documentation
Customer support
Community forum
Trust center
Partners
View all resources
Open Source
COMMUNITY
GitHub Sponsors
Fund open source developers
PROGRAMS
Security Lab
Maintainer Community
GitHub Stars
Archive Program
REPOSITORIES
Topics
Trending
Collections
Enterprise
ENTERPRISE SOLUTIONS
Enterprise platform
AI-powered developer platform
AVAILABLE ADD-ONS
GitHub Advanced Security
Enterprise-grade security features
Copilot for Business
Enterprise-grade AI features
Premium Support
Enterprise-grade 24/7 support
Pricing
Search
/
Sign in
Sign up
Appearance settings
You signed in with another tab or window.
Reload
to refresh your session.
You signed out in another tab or window.
Reload
to refresh your session.
You switched accounts on another tab or window.
Reload
to refresh your session.
Dismiss alert
{{ message }}
LabKey
/
MacCossLabModules
Public
Notifications
You must be signed in to change notification settings
Fork
2
Star
1
Code
Issues
0
Pull requests
1
Actions
Projects
Security and quality
0
Insights
Additional navigation options
Code
Issues
Pull requests
Actions
Projects
Security and quality
Insights
Fixed the private data reminder schedule and NCBI publication search failure handling
- #675
#675
Open
vagisha
wants to merge 24 commits into
release26.7-SNAPSHOT
LabKey/MacCossLabModules:release26.7-SNAPSHOT
from
26.7_fb_panoramapublic-reminder-fixes
LabKey/MacCossLabModules:26.7_fb_panoramapublic-reminder-fixes
Copy head branch name to clipboard
Conversation
Commits
24
(24)
Checks
Files changed
Open
Fixed the private data reminder schedule and NCBI publication search failure handling
#675
vagisha
wants to merge 24 commits into
release26.7-SNAPSHOT
LabKey/MacCossLabModules:release26.7-SNAPSHOT
from
26.7_fb_panoramapublic-reminder-fixes
LabKey/MacCossLabModules:26.7_fb_panoramapublic-reminder-fixes
Copy head branch name to clipboard
Commits
Commits on Aug 28, 2026
Fixed Panorama Public reminder scheduling and hardened NCBI publication search
Show description for e35411c
vagisha
committed
e35411c
View commit details
Copy full SHA for e35411c
Browse repository at this point
Captured PublicationSearchTest's server error-count baseline just before the bad-key search so incidental NCBI 5xx in earlier steps don't make checkExpectedErrors flaky on dev
vagisha
committed
02022c4
View commit details
Copy full SHA for 02022c4
Browse repository at this point
Asserted enablePublicationSearch round-trips through save in PanoramaPublicBaseTest's reminder-settings helper
vagisha
committed
21a0cc3
View commit details
Copy full SHA for 21a0cc3
Browse repository at this point
Scoped MockNcbiPublicationSearchService matching to the decoded db/term/id query params instead of substring-scanning the whole request URL
vagisha
committed
3c4bdfd
View commit details
Copy full SHA for 3c4bdfd
Browse repository at this point
Added unit tests for the getString retry loop (retry-then-succeed, give-up-after-MAX_HTTP_ATTEMPTS, no-retry-on-4xx); made executeGet protected so a test subclass can drive it
Show description for 0c2058f
vagisha
committed
0c2058f
View commit details
Copy full SHA for 0c2058f
Browse repository at this point
Commits on Aug 31, 2026
Protected the NCBI API key and made a failed publication search visible
Show description for 74370ce
vagisha
and
claude
committed
74370ce
View commit details
Copy full SHA for 74370ce
Browse repository at this point
Moved the NCBI API key coverage into its own Selenium test
Show description for 759f5d9
vagisha
and
claude
committed
759f5d9
View commit details
Copy full SHA for 759f5d9
Browse repository at this point
Commits on Sep 1, 2026
Hardened the NCBI retry loop against 429, interruption and unreadable error bodies
Show description for 2a67949
vagisha
and
claude
committed
2a67949
View commit details
Copy full SHA for 2a67949
Browse repository at this point
Fixed two date tests that failed on the last days of a month
Show description for 89677df
vagisha
and
claude
committed
89677df
View commit details
Copy full SHA for 89677df
Browse repository at this point
Checked the NCBI API key before the reminder job posts anything
Show description for 2bf6107
vagisha
and
claude
committed
2bf6107
View commit details
Copy full SHA for 2bf6107
Browse repository at this point
Made the retry loop's waiting overridable so tests can check the delays
Show description for 4981c02
vagisha
and
claude
committed
4981c02
View commit details
Copy full SHA for 4981c02
Browse repository at this point
Stopped a 429 from the API key check looking like a rejected key
Show description for 151992f
vagisha
and
claude
committed
151992f
View commit details
Copy full SHA for 151992f
Browse repository at this point
Commits on Sep 7, 2026
Addressed review findings on the NCBI publication search and reminder job
Show description for bf13cb4
vagisha
and
claude
committed
bf13cb4
View commit details
Copy full SHA for bf13cb4
Browse repository at this point
Commits on Sep 8, 2026
Reported the reminder job's status from what the run recorded, not from log levels
Show description for f95b5e2
vagisha
and
claude
committed
f95b5e2
View commit details
Copy full SHA for f95b5e2
Browse repository at this point
Commits on Sep 27, 2026
Fixed the NCBI API key settings form and retried connection resets
Show description for 77d8727
vagisha
and
claude
committed
77d8727
View commit details
Copy full SHA for 77d8727
Browse repository at this point
Stopped the reminder settings from requiring an encryption key
Show description for 6981aeb
vagisha
and
claude
committed
6981aeb
View commit details
Copy full SHA for 6981aeb
Browse repository at this point
Let Cancel in the pipeline UI stop the private data reminder job
Show description for 37a97f0
vagisha
and
claude
committed
37a97f0
View commit details
Copy full SHA for 37a97f0
Browse repository at this point
Addressed review findings on NCBI error handling and the reminder settings page
Show description for 34915b8
vagisha
and
claude
committed
34915b8
View commit details
Copy full SHA for 34915b8
Browse repository at this point
Rejected NCBI API key changes on a server with no encryption key
Show description for 303e446
vagisha
and
claude
committed
303e446
View commit details
Copy full SHA for 303e446
Browse repository at this point
Moved the NCBI HTTP request code into NcbiHttpClient
Show description for 45b4e79
vagisha
and
claude
committed
45b4e79
View commit details
Copy full SHA for 45b4e79
Browse repository at this point
Addressed code review findings on the NCBI API key and NcbiHttpClient
Show description for abfa8b0
vagisha
and
claude
committed
abfa8b0
View commit details
Copy full SHA for abfa8b0
Browse repository at this point
Commits on Oct 5, 2026
Added an NCBI check of the API key before it is saved
Show description for 3d30a1f
vagisha
and
claude
committed
3d30a1f
View commit details
Copy full SHA for 3d30a1f
Browse repository at this point
Changed the reminder settings test cleanup to restore settings through an API call
Show description for 6ccc96d
vagisha
and
claude
committed
6ccc96d
View commit details
Copy full SHA for 6ccc96d
Browse repository at this point
Added a stop to the reminder job's publication search when NCBI does not respond
Show description for 5c8af3d
vagisha
and
claude
committed
5c8af3d
View commit details
Copy full SHA for 5c8af3d
Browse repository at this point
You can’t perform that action at this time.