Skip to content

Navigation Menu

Explore
By company size
By use case
By industry
View all solutions
Topics
- AI
- DevOps
- Security
- Software Development
- View all
Explore
- GitHub Sponsors
  Fund open source developers
- The ReadME Project
  GitHub community articles
Repositories
- Enterprise platform
  AI-powered developer platform
Available add-ons
Pricing

Search code, repositories, users, issues, pull requests...

Search

Clear

Search syntax tips

Provide feedback

We read every piece of feedback, and take your input very seriously.

Include my email address so I can be contacted

Saved searches

Use saved searches to filter your results more quickly

Name

Query

To see all available qualifiers, see our documentation.

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

ThePalaceProject / circulation Public

Notifications You must be signed in to change notification settings
Fork 7
Star 6

Code
Pull requests 2
Actions
Security
Insights

Additional navigation options

Code
Pull requests
Actions
Security
Insights

PP-495 Analytics as a container service#1487

Merged

RishiDiwanTT merged 15 commits intomainThePalaceProject/circulation:mainfrom feature/analytics-serviceThePalaceProject/circulation:feature/analytics-serviceCopy head branch name to clipboard

Nov 6, 2023

+278-1,360

Conversation Commits15 (15)Checks Files changed

Commits

Commits on Oct 30, 2023

Analytics as a service through dependency injection
RishiDiwanTT
committed
Removed the admin UI analytics service controller
RishiDiwanTT
committed
Added an analytics container with a singleton so the analytics object is created only once
RishiDiwanTT
committed
Mypy fixes
RishiDiwanTT
committed

Commits on Oct 31, 2023

Switched to the injection pattern for CirculationManager load_settings
RishiDiwanTT
committed
Removed the analytics requirement from the licensing methods
RishiDiwanTT
committed
Inject s3 service directly in the the analytics provider
RishiDiwanTT
committed
Removed location_source and adjoining logic
RishiDiwanTT
committed
Moved away from container_isntance() towards injection
RishiDiwanTT
committed
Removed unused variables
RishiDiwanTT
committed

Commits on Nov 1, 2023

Removed the admin analytics route
RishiDiwanTT
committed
Injected analytics into the overdrive new title script
RishiDiwanTT
committed
Typing fixes
RishiDiwanTT
committed

Commits on Nov 2, 2023

Review comments update
RishiDiwanTT
committed
Removed the Serivces container from the CirculationManager
RishiDiwanTT
committed

Footer

© 2025 GitHub, Inc.

Footer navigation

Terms
Privacy
Security
Status
Docs
Contact

You can’t perform that action at this time.