Skip to content

Conversation

@Aerospace-prog
Copy link

@Aerospace-prog Aerospace-prog commented Apr 6, 2025

Pull Request: Implemented Scheme Manager Microservice

📌 Description

This pull request introduces a fully functional Scheme Manager microservice for the ONEST ecosystem. It handles scheme retrieval, application submission, and status tracking — meeting all acceptance criteria of issue #2 .


✅ Features Implemented

🔌 API Endpoints

Method Endpoint Description
POST /api/applications Submit a new application
GET /api/applications Get all applications
GET /api/applications/:id Get application status by ID
PUT /api/applications/:id/status Update application status

🧱 Project Structure

├── cmd/ # Application entrypoint
├── controllers/ # API handlers
├── models/ # Request & response models
├── database/ # DB setup & queries
├── routes/ # Routing layer
├── Dockerfile
├── docker-compose.yml
├── .env.example
└── README.md


⚙️ Tech Stack

  • Golang (Gin framework)
  • PostgreSQL
  • Docker & Docker Compose
  • RESTful APIs
  • Environment-based configuration
  • Custom logging

🧪 Testing

  • All endpoints tested using Postman & cURL
  • Sample test payloads added in README
  • Handles success, validation, and error responses gracefully

🐳 Run Locally

docker-compose up --build

🙏 Acknowledgements

Thanks to the ONEST team for this well-documented bounty opportunity.
Looking forward to feedback and merging this contribution!

🔧 Fixes

#2 (fixed)

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 12 out of 15 changed files in this pull request and generated 3 comments.

Files not reviewed (3)
  • scheme-manager/.env: Language not supported
  • scheme-manager/Dockerfile: Language not supported
  • scheme-manager/go.mod: Language not supported
Comments suppressed due to low confidence (1)

scheme-manager/config/db.go:30

  • The connection string includes DB_PORT but it is not defined in docker-compose.yml, which might result in an incorrect or empty value. Consider defining DB_PORT or handling default port conditions.
os.Getenv("DB_PORT"),

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C4GT Coding C4GT Coding C4GT Community C4GT Community

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants