Skip to content

Conversation

@Fluder-Paradyne
Copy link
Collaborator

Description

Add terminal to workbench

Related Issues

Solution and Design

Test Plan

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Docs update

Checklist

  • My pull request is atomic and focuses on a single change.
  • I have documented my changes clearly and comprehensively.
  • I have added the required tests.

@Fluder-Paradyne Fluder-Paradyne changed the title Terminal poc Terminal for workbench Jul 18, 2024
github.com/google/go-github v17.0.0+incompatible
github.com/google/uuid v1.4.0
github.com/googollee/go-socket.io v1.7.0
github.com/gorilla/websocket v1.5.3
Copy link
Collaborator

Choose a reason for hiding this comment

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

why gorialla we had preferred socketio over it

Copy link
Collaborator

Choose a reason for hiding this comment

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

https://github.com/googollee/go-socket.io we had moved ahead with socketio for better lifecycle and rejected this @czezi

github.com/google/go-github v17.0.0+incompatible
github.com/google/uuid v1.4.0
github.com/googollee/go-socket.io v1.7.0
github.com/gorilla/websocket v1.5.3
Copy link
Collaborator

Choose a reason for hiding this comment

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

https://github.com/googollee/go-socket.io we had moved ahead with socketio for better lifecycle and rejected this @czezi

github.com/emirpasic/gods v1.18.1 // indirect
github.com/gabriel-vasile/mimetype v1.4.3 // indirect
github.com/gin-contrib/sse v0.1.0 // indirect
github.com/gin-contrib/zap v1.1.3 // indirect
Copy link
Collaborator

Choose a reason for hiding this comment

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

why are package getting removed?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

they are just moved up

github.com/golang/protobuf v1.5.4 // indirect
github.com/gomodule/redigo v1.9.2 // indirect
github.com/google/go-querystring v1.1.0 // indirect
github.com/gorilla/websocket v1.5.3 // indirect
Copy link
Collaborator

Choose a reason for hiding this comment

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

why is this removed?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

they are just moved up

r.Use(TerminalGinMiddleware("http://localhost:3000, https://developer.superagi.com"))
r.RedirectTrailingSlash = false

api := r.Group("/api")
Copy link
Collaborator

Choose a reason for hiding this comment

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

I guess we should structure inside gateways all the websocket like in the other use cases @czezi what do you say?

@luciferlinx101 luciferlinx101 mentioned this pull request Jul 25, 2024
7 tasks
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.

4 participants