feat: migrate backend from beego to gin by vishalm0509 · Pull Request #335 · datazip-inc/olake-ui

vishalm0509 · 2026-03-09T13:57:42Z

Description

Replaced Beego HTTP stack with Gin across handlers, routing, middleware, and server startup while keeping the existing API contract and file structure.
Migrated ORM layer from Beego ORM to GORM, including DB initialization, model/query rewrites, and runtime automigration with existing schema compatibility.

Fixes # (issue)

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 not work as expected)
This change requires a documentation update

How Has This Been Tested?

Scenario A
Scenario B

Screenshots or Recordings

Related PR's (If Any):

Signed-off-by: badalprasadsingh <badal@datazip.io>

vikaxsh · 2026-03-10T10:48:10Z

-		if err != nil || value == "" {
-			panic("Required config variable not found: ," + v)
+func checkForRequiredVariables() {
+	cfg := appconfig.Load()


Init() calls appconfig.Load() at line 117 and checkForRequiredVariables() also calls here, pass cfg in the function

Config is loaded once at startup and then every time we call appconfig.Load() it returns the already loaded config. Therefore, no re-computation of all the envs.

// this loads first on startup var cfg = loadConfig() // this serves the already loaded cfg func Load() Config { return cfg }

vikaxsh · 2026-03-11T05:53:54Z

+	gormlogger "gorm.io/gorm/logger"

-	"github.com/datazip-inc/olake-ui/server/internal/constants"
+	"github.com/datazip-inc/olake-ui/server/internal/appconfig"


how can we avoid import of appconfig in multiple packages

Config is loaded once at startup and then every time we call appconfig.Load() it returns the already loaded config. Therefore, no re-computation of all the envs.

// this loads first on startup var cfg = loadConfig() // this serves the already loaded cfg func Load() Config { return cfg }

It is similar to that of web.AppConfig.String(constants.ConfOLakePostgresUser) we import web everywhere. appconfig is similar to that

can we think of any alternate to avoid this now

We have decided to use this now

vikaxsh · 2026-03-11T07:39:38Z

 )

-// ValidateStruct validates any struct that has `validate` tags.
-func Validate(s interface{}) error {


don't we require this anymore?

No, here c.ShouldBindJSON() handles bindings.

func bindAndValidate(c *gin.Context, target interface{}) error { return c.ShouldBindJSON(target) }

We have replaced Username string json:"username" validate:"required" example:"admin" (validate) with this (binding) which does the same work.

vikaxsh · 2026-03-11T08:13:11Z

-	SessionData   string    `json:"session_data" orm:"column(session_data);type(text)"`
-	SessionExpiry time.Time `json:"session_expiry" orm:"column(session_expiry)"`
+	SessionKey    string    `json:"session_key" gorm:"column:session_key;primaryKey;size:64"`
+	SessionData   []byte    `json:"session_data" gorm:"column:session_data;type:bytea"`


why? Session.SessionData TEXT -> BYTEA

In the earlier code beego also wrote data to postgres in []byte

// Add session table if sessions are enabled if web.BConfig.WebConfig.Session.SessionOn { _, err = orm.NewOrm().Raw(`CREATE TABLE IF NOT EXISTS session ( session_key VARCHAR(64) PRIMARY KEY, session_data BYTEA, session_expiry TIMESTAMP WITH TIME ZONE );`).Exec()

vikaxsh · 2026-03-11T08:15:26Z

+	BaseModel
+	ID               int     `json:"id" gorm:"column:id;primaryKey;autoIncrement"`
+	Name             string  `json:"name" gorm:"column:name;size:100"`
+	SourceID         int     `json:"source_id" gorm:"column:source_id"`


won't it cause any issue changing *Source -> int

This is FK reference ID, we still can access the Source object with preLoad

Source *Source `json:"source,omitempty" gorm:"foreignKey:SourceID;references:ID"` Destination *Destination `json:"destination,omitempty" gorm:"foreignKey:DestID;references:ID"` CreatedBy *User `json:"created_by,omitempty" gorm:"foreignKey:CreatedByID;references:ID"` UpdatedBy *User `json:"updated_by,omitempty" gorm:"foreignKey:UpdatedByID;references:ID"`

vikaxsh · 2026-03-11T17:11:55Z

 // @Router /signup [post]
-func (h *Handler) Signup() {
+func (h *Handler) Signup(c *gin.Context) {
 	var req models.User


can we create dto's for these as well?

vikaxsh · 2026-03-11T17:16:00Z

 		return
 	}
-	utils.SuccessResponse(&h.Controller, "Destinations listed successfully", items)
+	successResponse(c, "Destinations listed successfully", items)


Suggested change

successResponse(c, "Destinations listed successfully", items)

successResponse(c, "destinations listed successfully", items)

vikaxsh · 2026-03-11T17:19:44Z

 		return
 	}
-
-	if req.Source.ID == nil {
-		if err := dto.ValidateSourceType(req.Source.Type); err != nil {


don't we require source type validation anymore?

vikaxsh · 2026-03-11T17:21:24Z

-			utils.ErrorResponse(&h.Controller, http.StatusBadRequest, fmt.Sprintf("failed to validate request: %s", err), err)
-			return
-		}
-		if req.Source.Name == "" || req.Source.Version == "" || req.Source.Config == "" {


don't we require this check anymore?

vikaxsh · 2026-03-11T17:22:45Z

-	h.Ctx.Output.Header("Access-Control-Expose-Headers", "Content-Disposition")
-
-	if err := h.etl.StreamLogArchive(id, filePath, h.Ctx.ResponseWriter); err != nil {
-		logger.Errorf("failed to stream log archive job_id[%d]: %s", id, err)


add this back

Signed-off-by: badalprasadsingh <badal@datazip.io>

* initial commit Signed-off-by: badalprasadsingh <badal@datazip.io> * lint fix Signed-off-by: badalprasadsingh <badal@datazip.io> * major backend APIs for fusion Signed-off-by: badalprasadsingh <badal@datazip.io> * refactor: domain-driver level Signed-off-by: badalprasadsingh <badal@datazip.io> * major refactoring Signed-off-by: badalprasadsingh <badal@datazip.io> * chore: services and add cancel api Signed-off-by: badalprasadsingh <badal@datazip.io> * fix: cron Signed-off-by: badalprasadsingh <badal@datazip.io> * add go.sum Signed-off-by: badalprasadsingh <badal@datazip.io> * minor lint fixes Signed-off-by: badalprasadsingh <badal@datazip.io> * fix: compaction.go Signed-off-by: badalprasadsingh <badal@datazip.io> * minor lint Signed-off-by: badalprasadsingh <badal@datazip.io> * add compaction enable Signed-off-by: badalprasadsingh <badal@datazip.io> * fix: minor Signed-off-by: badalprasadsingh <badal@datazip.io> * sort databases and catalogs Signed-off-by: badalprasadsingh <badal@datazip.io> * minor Signed-off-by: badalprasadsingh <badal@datazip.io> * fix: minor Signed-off-by: badalprasadsingh <badal@datazip.io> * removed unceessary dependencies Signed-off-by: badalprasadsingh <badal@datazip.io> * minor Signed-off-by: badalprasadsingh <badal@datazip.io> * minor Signed-off-by: badalprasadsingh <badal@datazip.io> * fix: minor Signed-off-by: badalprasadsingh <badal@datazip.io> * major refactoring Signed-off-by: badalprasadsingh <badal@datazip.io> * fix: remove OptimizerGroup from constants Signed-off-by: badalprasadsingh <badal@datazip.io> * minor Signed-off-by: badalprasadsingh <badal@datazip.io> * minor Signed-off-by: badalprasadsingh <badal@datazip.io> * minor Signed-off-by: badalprasadsingh <badal@datazip.io> * minor Signed-off-by: badalprasadsingh <badal@datazip.io> * minor Signed-off-by: badalprasadsingh <badal@datazip.io> * optimisation -> optimization Signed-off-by: badalprasadsingh <badal@datazip.io> * optimisation -> optimization Signed-off-by: badalprasadsingh <badal@datazip.io> * minor Signed-off-by: badalprasadsingh <badal@datazip.io> * refresh token logic Signed-off-by: badalprasadsingh <badal@datazip.io> * update piggy backing to add sep download logic Signed-off-by: badalprasadsingh <badal@datazip.io> * review changes Signed-off-by: badalprasadsingh <badal@datazip.io> * minor change Signed-off-by: badalprasadsingh <badal@datazip.io> * minor change Signed-off-by: badalprasadsingh <badal@datazip.io> * minor change Signed-off-by: badalprasadsingh <badal@datazip.io> * minor change Signed-off-by: badalprasadsingh <badal@datazip.io> * minor change Signed-off-by: badalprasadsingh <badal@datazip.io> * minor change Signed-off-by: badalprasadsingh <badal@datazip.io> * minor todi Signed-off-by: badalprasadsingh <badal@datazip.io> * adding todo for camel case Signed-off-by: badalprasadsingh <badal@datazip.io> * adding todo for camel or snake case Signed-off-by: badalprasadsingh <badal@datazip.io> * fix: minor Signed-off-by: badalprasadsingh <badal@datazip.io> * apply insert filter on all apis Signed-off-by: badalprasadsingh <badal@datazip.io> * chore: cleanup * import feature Signed-off-by: badalprasadsingh <badal@datazip.io> * minor Signed-off-by: badalprasadsingh <badal@datazip.io> * feat(optimization): import destination as catalog in opt (#343) * import feature Signed-off-by: badalprasadsingh <badal@datazip.io> * minor Signed-off-by: badalprasadsingh <badal@datazip.io> --------- Signed-off-by: badalprasadsingh <badal@datazip.io> Co-authored-by: Ankit Sharma <111491139+hash-data@users.noreply.github.com> * review comments Signed-off-by: badalprasadsingh <badal@datazip.io> * fix: minor Signed-off-by: badalprasadsingh <badal@datazip.io> * fix: lint Signed-off-by: badalprasadsingh <badal@datazip.io> * fix: lint Signed-off-by: badalprasadsingh <badal@datazip.io> * fix: minor lint issues * import feature Signed-off-by: badalprasadsingh <badal@datazip.io> * minor Signed-off-by: badalprasadsingh <badal@datazip.io> * Merge pull request #346 from datazip-inc/fix/trivy_docker fix: suppress CVE-2026-34040 in Trivy until testcontainers-go upgrade * review comments Signed-off-by: badalprasadsingh <badal@datazip.io> * fix: minor Signed-off-by: badalprasadsingh <badal@datazip.io> * fix: lint Signed-off-by: badalprasadsingh <badal@datazip.io> * fix: lint Signed-off-by: badalprasadsingh <badal@datazip.io> --------- Signed-off-by: badalprasadsingh <badal@datazip.io> Co-authored-by: vishal-datazip <vishal@datazip.io> Co-authored-by: Ankit Sharma <111491139+hash-data@users.noreply.github.com> * fix: minor Signed-off-by: badalprasadsingh <badal@datazip.io> * fix: minor Signed-off-by: badalprasadsingh <badal@datazip.io> * fix: minor lint issues Signed-off-by: badalprasadsingh <badal@datazip.io> * fix: minor Signed-off-by: badalprasadsingh <badal@datazip.io> * minor Signed-off-by: badalprasadsingh <badal@datazip.io> * review changes + lock Signed-off-by: badalprasadsingh <badal@datazip.io> * fix: lint Signed-off-by: badalprasadsingh <badal@datazip.io> * fix: minor Signed-off-by: badalprasadsingh <badal@datazip.io> --------- Signed-off-by: badalprasadsingh <badal@datazip.io> Co-authored-by: vishalm0509 <vishal@datazip.io> Co-authored-by: Ankit Sharma <111491139+hash-data@users.noreply.github.com>

…c/olake-ui into feat/gin

…o feat/gin

vikaxsh · 2026-03-13T09:35:26Z

-	// Project settings routes
-	web.Router("/api/v1/project/:projectid/settings", h, "put:UpsertProjectSettings")
-	web.Router("/api/v1/project/:projectid/settings", h, "get:GetProjectSettings")
-
-	// validation routes
-	web.Router("/api/v1/project/:projectid/check-unique", h, "post:CheckUniqueName")
-
-	// platform routes


add these comments back

vikaxsh · 2026-03-13T09:38:13Z

+	CopyRequestBody       bool
+	MaxMemory             int64
+	MaxUploadSize         int64


can we remove CopyRequestBody

vikaxsh · 2026-03-13T09:50:00Z

+	return projectID, nil
+}
+
+func getIDParam(c *gin.Context, key string) (int, error) {


key always will be "id" here, can we avoid passing in function

Right, but for other modules will it still be id always ? since we are using this as a global util for all the modules

I have made changes anyway since the name is explicitly GetIDParam but the above is something we should think about

vikaxsh · 2026-04-10T08:33:36Z

 }

 // GetAll retrieves all jobs
 func (db *Database) ListJobs() ([]*models.Job, error) {


Yes, removed.

vikaxsh · 2026-04-10T08:37:51Z

 	var users []*models.User
-	_, err := db.ormer.QueryTable(constants.TableNameMap[constants.UserTable]).All(&users)
+	err := db.conn.Find(&users).Error
 	return users, err


fmt.Errorf("failed to find user: %s", err)

here, err can be nil as well. This is the design we are following in the database layer.

We are adding a proper error message in the service layer to which this is appended

vikaxsh · 2026-04-10T09:39:26Z

+	c.Status(http.StatusOK)
+	if err := h.etl.StreamLogArchive(id, filePath, c.Writer); err != nil {


what if StreamLogArchive returns an error without writing any bytes, it would return 200 OK with an empty response

Yes, made changes

vikaxsh · 2026-04-10T12:19:47Z

+	}
+}
+
+func (s *Server) Engine() *gin.Engine {


where are we using this?

removed, was using it earlier

vikaxsh · 2026-04-10T12:28:25Z

 // User-related methods on AppService

-func (s Service) CreateUser(_ context.Context, req *models.User) error {
+func (s *Service) CreateUser(_ context.Context, req *models.User) error {


why pointer receiver?

made changes

vikaxsh · 2026-04-11T09:48:40Z

-	// platform routes
-	web.Router("/api/v1/platform/releases", etlHandler, "get:GetReleaseUpdates")
-	web.Router("/api/v1/platform/opt/status", h, "get:GetoptimizationStatus")
+	etl := engine.Group("/api/v1")


in beego we were applying auth for /api/* path

// Apply auth middleware to protected routes web.InsertFilter("/api/*", web.BeforeRouter, middleware.AuthMiddleware)

Auth middlware only applied to routes with prefix /api

yes
now it is applying to -> etl := engine.Group("/api/v1"),etl.Use(h.AuthMiddleware())

Yes, made changes

…o feat/gin

vikaxsh · 2026-04-14T07:24:54Z

 	source.Config = eConfig
-	_, err = db.ormer.Update(source)
-	return err
+	return db.conn.Updates(source).Error


gorm.Updates() Silently Skips Zero-Values on Structs
please check following doc
https://gorm.io/docs/update.html#Updates-multiple-columns

None of these can be zero value, because in the BindAndValidate we are mentioning the fields as required

still we should use correct function

vikaxsh · 2026-04-14T07:31:45Z

+	v := viper.New()
+
+	// Note: config priority: env variables -> file (app.yaml)
+	v.SetConfigFile("./conf/app.yaml")


can it cause any issue in k8s where all configs come from env vars

If env is set then we prioritize env over file

vikaxsh · 2026-04-14T07:45:35Z

+		return fmt.Errorf("failed to marshal session payload: %w", err)
+	}
+
+	err = s.db.UpsertSession(sessionID, payload, expiresAt)


why upsert not create

Create makes sense here. Made changes

vikaxsh · 2026-04-14T08:06:33Z

-	// platform routes
-	web.Router("/api/v1/platform/releases", etlHandler, "get:GetReleaseUpdates")
-	web.Router("/api/v1/platform/opt/status", h, "get:GetoptimizationStatus")
+	etl := engine.Group("/api/v1")


yes
now it is applying to -> etl := engine.Group("/api/v1"),etl.Use(h.AuthMiddleware())

vikaxsh · 2026-04-14T08:17:47Z

+func configureMode(cfg *appconfig.Config) {
+	switch cfg.RunMode {
+	case "localdev":
+		gin.SetMode(gin.DebugMode)


gin.SetMode() is a global setter that must be called before any gin.New() or gin.Default() call. Calling it after the engine is already created and middleware is already registered changes the global mode but has no effect on the already-constructed engine.

It will not affect the code. But it's mentioned as good practice. Moved it to the top

vikaxsh · 2026-04-14T08:21:03Z

+	configureMode(cfg)
+	configureBaseRoutes(engine)
+
+	if cfg.RunMode == "localdev" {


we are checking runmode redundantly can we avoid this and check run mode at one place for all ops

We are just reading the value from the already loaded config. All these are separate operations and cannot be grouped together.

vikaxsh · 2026-04-14T08:24:06Z

 	if h.Optimization != nil {
 		optHandler := h.Optimization
+		opt := engine.Group("/api/opt/v1")
+		opt.Use(h.AuthMiddleware())


h.AuthMiddleware() is called twice

It was called twice because we had 2 separate route groups for ETL and OPT. Now we are using a root API group where I have applied the middleware. Solved

vikaxsh · 2026-04-14T08:26:42Z


-		// piggy backing
-		web.Router("/api/opt/v1/*", optHandler, "get:PiggyBacking;post:PiggyBacking;put:PiggyBacking;delete:PiggyBacking")
+type ModuleNoRouteHandler struct {


can we move the struct and function in httpserver folder

Yes we can move it. DOne

The noModuleConfiguration is better if it's in router.go as it's related to that. Move the actual implementation to httpserver package.

vikaxsh · 2026-04-14T08:28:33Z

+		// Register optimization as a module fallback for unmatched /api/opt/v1/*.
+		// This avoids route tree conflicts from wildcard catch-all registration.
+		moduleHandlers = append(moduleHandlers, routes.ModuleNoRouteHandler{
+			PathPrefix: "/api/opt/v1/",


we should place all the routes together in router.go

vikaxsh · 2026-04-14T08:35:16Z

+
+const frontendDistPath = "/opt/frontend/dist"
+
+type Server struct {


it would be better if we create all the functions with server as receiver

…sion)

badalprasadsingh and others added 13 commits March 2, 2026 18:12

refactor: etl handlers

8ba3dc1

Signed-off-by: badalprasadsingh <badal@datazip.io>

initial commit

9b33fab

Signed-off-by: badalprasadsingh <badal@datazip.io>

lint fix

2283d86

Signed-off-by: badalprasadsingh <badal@datazip.io>

major backend APIs for fusion

2a8b712

Signed-off-by: badalprasadsingh <badal@datazip.io>

refactor: domain-driver level

e1d9715

Signed-off-by: badalprasadsingh <badal@datazip.io>

chore: move config management away from beego

83b0118

chore: setup gin

1dce1de

chore: migrate server and handlers to gin

62c8916

major refactoring

e359dff

Signed-off-by: badalprasadsingh <badal@datazip.io>

feat: migrate beego-orm to gorm

90fac51

fix: proper error code

ddbdf10

chore: version upgrade

a6a2342

chore: beego cleanup

978a175

vishalm0509 changed the title ~~feat: migrate from beego to gin~~ feat: migrate backend from beego to gin Mar 9, 2026

chore: services and add cancel api

176cf6b

Signed-off-by: badalprasadsingh <badal@datazip.io>

vishalm0509 changed the base branch from master to staging March 9, 2026 18:33

vishalm0509 added 3 commits March 10, 2026 00:17

chore: update CI to read go-version from mod

b86178e

fix: install golangci-lint using goinstall instead of binary

3027ebc

chore: final cleanup

4c2ba45

vishalm0509 marked this pull request as ready for review March 9, 2026 21:15

vishalm0509 and others added 5 commits March 10, 2026 03:35

chore: fix lint

bfa257d

fix: cron

95e2eea

Signed-off-by: badalprasadsingh <badal@datazip.io>

add go.sum

8dcc0d5

Signed-off-by: badalprasadsingh <badal@datazip.io>

chore: revert compose changes

cad2a9d

minor lint fixes

933a2e9

Signed-off-by: badalprasadsingh <badal@datazip.io>

vikaxsh reviewed Mar 11, 2026

View reviewed changes

badalprasadsingh added 4 commits March 12, 2026 06:44

fix: compaction.go

29c8085

Signed-off-by: badalprasadsingh <badal@datazip.io>

minor lint

8eb739b

Signed-off-by: badalprasadsingh <badal@datazip.io>

minor refactoring fix

9dbdece

Signed-off-by: badalprasadsingh <badal@datazip.io>

minor lint

da1a672

Signed-off-by: badalprasadsingh <badal@datazip.io>

vishalm0509 and others added 2 commits April 6, 2026 14:37

fix: utils import error

2263ec1

Base automatically changed from feat/fusion to refactor/etl-handlers April 6, 2026 15:14

vishalm0509 added 2 commits April 7, 2026 12:08

Merge branch 'refactor/etl-handlers' of https://github.com/datazip-in…

42d27b1

…c/olake-ui into feat/gin

fix: merge errors

6cb8244

Base automatically changed from refactor/etl-handlers to staging April 7, 2026 09:42

vishalm0509 added 11 commits April 7, 2026 19:09

chore: minor changes

ea45630

fix: use no-group to fix route conflicts in router.go

1be6085

Merge branch 'staging' of https://github.com/datazip-inc/olake-ui int…

6cd63f1

…o feat/gin

chore: handle no-route handler

49f5837

chore: self review changes

bd75bf6

Merge branch 'staging' into feat/gin

64355d0

fix: ci lint error

234e44f

fix: ci errors

54db03f

Merge branch 'staging' of https://github.com/datazip-inc/olake-ui int…

ae95d1a

…o feat/gin

fix: optimization status endpoint

a90b1eb

Merge branch 'staging' into feat/gin

6f8c311

vikaxsh reviewed Apr 11, 2026

View reviewed changes

vishalm0509 added 5 commits April 13, 2026 18:36

chore: remove copyrequestbody

068ac7b

chore: resolve comments

8119ead

chore: remove dead code

216f227

Merge branch 'staging' of https://github.com/datazip-inc/olake-ui int…

4918861

…o feat/gin

chore: remove unused method

c2707c3

vikaxsh reviewed Apr 14, 2026

View reviewed changes

vishalm0509 added 6 commits April 14, 2026 15:30

chore: add middleware to root api group

2b79fdf

chore: add session cleanup

4d9d1ba

chore: cleanup NewHandler

648da1a

chore: refactor no_route_handler and update functions to methods (ses…

bb2ba7f

…sion)

chore: update zero values as well

7968dd6

chore: add TODO for session_on

6aa0db1

	successResponse(c, "Destinations listed successfully", items)
	successResponse(c, "destinations listed successfully", items)

		c.Status(http.StatusOK)
		if err := h.etl.StreamLogArchive(id, filePath, c.Writer); err != nil {


		const frontendDistPath = "/opt/frontend/dist"

		type Server struct {

Conversation

vishalm0509 commented Mar 9, 2026

Description

Type of change

How Has This Been Tested?

Screenshots or Recordings

Related PR's (If Any):

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

vishalm0509 Apr 13, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

vishalm0509 Apr 13, 2026 •

edited

Loading