Skip to content

Conversation

@AnishSarkar22
Copy link
Contributor

@AnishSarkar22 AnishSarkar22 commented Jan 25, 2026

Description

  • Git is added to Docker all in one and backend Dockerfile for fixing github connector.
  • Removed status json for github connector.

Motivation and Context

FIX #

Screenshots

API Changes

  • This PR includes API changes

Change Type

  • Bug fix
  • New feature
  • Performance improvement
  • Refactoring
  • Documentation
  • Dependency/Build system
  • Breaking change
  • Other (specify):

Testing Performed

  • Tested locally
  • Manual/QA verification

Checklist

  • Follows project coding standards and conventions
  • Documentation updated as needed
  • Dependencies updated as needed
  • No lint/build errors or new warnings
  • All relevant tests are passing

High-level PR Summary

This PR fixes the GitHub connector functionality by ensuring the git binary is available in the Docker containers. The change adds git as a dependency in the backend Dockerfile, prevents it from being removed in the all-in-one Dockerfile, and removes the warning status configuration for the GitHub connector indicating that the indexing issue has been resolved.

⏱️ Estimated Review Time: 5-15 minutes

💡 Review Order Suggestion
Order File Path
1 surfsense_web/components/assistant-ui/connector-popup/config/connector-status-config.json
2 surfsense_backend/Dockerfile
3 Dockerfile.allinone

Need help? Join our Discord

Analyze latest changes

@vercel
Copy link

vercel bot commented Jan 25, 2026

@AnishSarkar22 is attempting to deploy a commit to the Rohan Verma's projects Team on Vercel.

A member of the Team first needs to authorize it.

@AnishSarkar22 AnishSarkar22 marked this pull request as ready for review January 25, 2026 21:56
Copy link

@recurseml recurseml bot left a comment

Choose a reason for hiding this comment

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

Review by RecurseML

🔍 Review performed on 13fdb3e..0e26fb1

✨ No bugs found, your code is sparkling clean

✅ Files analyzed, no issues (3)

Dockerfile.allinone
surfsense_backend/Dockerfile
surfsense_web/components/assistant-ui/connector-popup/config/connector-status-config.json

@MODSetter MODSetter merged commit b031b98 into MODSetter:dev Jan 26, 2026
8 of 12 checks passed
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