Skip to content

fix(bi-dashboard): add BiJob RPCs and enhance Dashboard features#35

Merged
ilramdhan merged 2 commits into
mutugading:mainfrom
ilramdhan:feat/bi-foundation
May 31, 2026
Merged

fix(bi-dashboard): add BiJob RPCs and enhance Dashboard features#35
ilramdhan merged 2 commits into
mutugading:mainfrom
ilramdhan:feat/bi-foundation

Conversation

@ilramdhan

Copy link
Copy Markdown
Member

Description

This pull request expands the BI (Business Intelligence) proto definitions to support featured dashboards and adds full CRUD (Create, Read, Update, Delete) support for BI jobs. The changes introduce new fields and RPC methods to improve dashboard management and enable dynamic ETL job operations.

Change Type

  • ✨ New service/message
  • ➕ Add field/RPC/enum value
  • 🔄 Modify validation
  • 📝 Documentation update
  • ⚠️ Deprecation
  • 🔧 Config/script changes

Proto Files Changed

  • finance/v1/bi.proto
  • Other: _______________

Changes Made

Dashboard enhancements:

  • Added is_featured and feature_order fields to the Dashboard message, allowing dashboards to be marked and ordered as featured. These fields are also included in the UpdateDashboardRequest message. [1] [2]
  • Introduced new RPCs and messages for listing featured dashboards: ListFeaturedDashboardsRequest, ListFeaturedDashboardsResponse, and the ListFeaturedDashboards RPC in the DashboardService. [1] [2]

BI job management:

  • Added messages for creating, updating, and deleting BI jobs: CreateBiJobRequest, CreateBiJobResponse, UpdateBiJobRequest, UpdateBiJobResponse, DeleteBiJobRequest, and DeleteBiJobResponse, with appropriate validation rules.
  • Added corresponding RPCs to the BiJobService for registering, updating, and soft-deleting BI jobs, with REST endpoint mappings.

Pre-merge Checklist

  • I have read and followed RULES.md
  • buf format -w applied
  • buf lint passes
  • buf breaking passes
  • Comments document new messages/fields
  • REST mappings follow conventions
  • Validation rules are complete
  • Field numbers are logical

Impact Assessment

  • Backend code regeneration required
  • Frontend code regeneration required
  • OpenAPI spec regeneration required

ilramdhan and others added 2 commits May 29, 2026 06:52
…Dashboards RPC

- Dashboard message: fields 25 (is_featured) and 26 (feature_order)
- UpdateDashboardRequest: optional is_featured/feature_order for pin/unpin
- New ListFeaturedDashboards RPC: GET /api/v1/finance/bi/dashboards/featured

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@ilramdhan ilramdhan added this to the Costing Release Milestone milestone May 31, 2026
@ilramdhan ilramdhan self-assigned this May 31, 2026
Copilot AI review requested due to automatic review settings May 31, 2026 05:26
@ilramdhan ilramdhan added the feat label May 31, 2026
@ilramdhan ilramdhan changed the title Add BiJob RPCs and enhance Dashboard features fix(bi-dashboard): add BiJob RPCs and enhance Dashboard features May 31, 2026
@ilramdhan ilramdhan moved this from Todo to In progress in Goapps Roadmap [Shared-Proto] May 31, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@ilramdhan ilramdhan merged commit f51a47d into mutugading:main May 31, 2026
1 check passed
@github-project-automation github-project-automation Bot moved this from In progress to Done in Goapps Roadmap [Shared-Proto] May 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants