Feat/gh 73/crowdactions

CollActionteam / collaction_backend

Backend code for CollAction

6 stars 4 forks source link

Feat/gh 73/crowdactions #119

Closed edreinoso closed 2 years ago

edreinoso commented 2 years ago

Included crowdaction endpoint functionality in the following structure:

handler/crowdaction.go
internals/crowdaction.go
models/crowdaction.go | commitment.go
repository/aws/dynamo.go
repository/aws/crowdactionManager.go

vrazdalovschi commented 2 years ago

Hi, I am not familiar with the whole project, but why do you use a lot of fmt.Print? Like: fmt.Println("GetCrowdactionsByStatus", status, startFrom)

Is it part of debug logs or for local testing during development? If it is done for log purposes, then all these prints must be replaced with a logger.

vrazdalovschi commented 2 years ago

@edreinoso could you please rebase your branch to reduce MERGE commits in your PR?