Skip to content

LFX coding challenge(assets)#190

Closed
Ao-chuba wants to merge 2 commits intoopencost:mainfrom
Ao-chuba:feature/assets-page-implementation
Closed

LFX coding challenge(assets)#190
Ao-chuba wants to merge 2 commits intoopencost:mainfrom
Ao-chuba:feature/assets-page-implementation

Conversation

@Ao-chuba
Copy link
Copy Markdown

@Ao-chuba Ao-chuba commented Feb 7, 2026

What does this PR change?

  • Implements a comprehensive Assets Page for granular cost analysis
  • Adds Time Window Selector (Today, 7d, 30d, 60d, 90d)
  • Adds Asset Type Filtering (dynamic filter for Nodes, Disks, Network, etc.)
  • Adds Interactive Visualizations with 3 chart types (Pie, Bar, Line)
  • Adds KPI Summary Cards with cost trend indicators (up/down arrows)
  • Adds Data Table with search, column sorting, and pagination
  • Adds CSV Export functionality for efficient data reporting
  • Implements production-ready patterns:
    • Smart fallback for trend indicators (uses real data when available)
    • Environment-based mock/real data toggle
    • Responsive layout (Carbon Design System)

Does this PR relate to any other PRs?

  • No

How will this PR impact users?

  • Users can now inspect infrastructure costs at the individual asset level.
  • Enables analyzing cost trends over specific time windows to identify spikes.
  • Provides multiple views (charts vs. tables) to visualize cost distribution.
  • "Export to CSV" allows for external reporting and analysis.

Does this PR address any GitHub or Zendesk issues?

How was this PR tested?

  • Verified with mock data matching the OpenCost API specification.
  • Audited for real data compatibility (all components map to real field names).
  • Tested responsive behavior on desktop and mobile viewports.
  • Verified logic for all filters, charts, and export functions.

Does this PR require changes to documentation?

  • No

Have you labeled this PR and its corresponding Issue as "next release" if it should be part of the next OpenCost release? If not, why not?

  • No, I do not have write access to the repository to add labels. This PR is a submission for the LFX Mentorship Challenge (Assets Page Implementation)

##ScreenShots
Screenshot 2026-02-07 013545
Screenshot 2026-02-07 013551
Screenshot 2026-02-07 013633
Screenshot 2026-02-07 152843

@netlify
Copy link
Copy Markdown

netlify Bot commented Feb 7, 2026

Deploy Preview for opencost-ui ready!

Name Link
🔨 Latest commit f862487
🔍 Latest deploy log https://app.netlify.com/projects/opencost-ui/deploys/69876fda2b01d30008710542
😎 Deploy Preview https://deploy-preview-190--opencost-ui.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@Ao-chuba Ao-chuba force-pushed the feature/assets-page-implementation branch from 42c63bb to 4ec6999 Compare February 7, 2026 10:26
Signed-off-by: aochuba <aochuba@example.com>
Signed-off-by: aochuba <aochuba52@gmail.com>
Signed-off-by: aochuba <aochuba@example.com>
@Ao-chuba Ao-chuba force-pushed the feature/assets-page-implementation branch from 4ec6999 to f862487 Compare February 7, 2026 17:01
@ameijer
Copy link
Copy Markdown
Member

ameijer commented Mar 2, 2026

Thank you for your application to our Term 1 Mentorship! We had so many amazing submissions that met and exceeded all our application criteria; this was the hardest mentee selection process we have had to go through. This means that there were many excellent applicants who did great work and correctly implemented the programming challenge, but we could only ultimately select one applicant. Unfortunately, were were unable to move forward with your application.

We hope you will consider applying to one of our future mentorship terms! in the meantime, we are always grateful for any contributions you want to make to OpenCost!.

Thanks again,
Alex, on behalf of the OpenCost Mentors

@ameijer ameijer closed this Mar 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support Assets in the UI

2 participants