SSWConsulting / SSWConsulting/TimePro.Tools

MCP Apps: leave balances import review (unmatched employees and warnings)

Open
#85 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Area: MCP Apps Priority: P3 Type: Feature
Dominant language
C#
Stars
3
Forks
0
Avg merge
29m
Merged PRs (30d)
28

Description

Part of MCP Apps (https://github.com/SSWConsulting/TimePro.Tools/issues/78). Write, accounting feature pack only, leave admins.

After import_leave_balances (path-based, destructive, host approval required), render the result: matched count, unmatchedEmployees with the Xero name that failed to match, warnings for implausible balances, and the mayHaveBeenApplied uncertain-write state with its recovery text. Read-only over the tool result; a re-import is a new tool call through the host prompt, never a page-side retry.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start at the MCP Apps entry point for import_leave_balances and inspect the tool result fields before rendering. Done means the page shows matched count, unmatchedEmployees, warnings, and mayHaveBeenApplied recovery text read-only, with re-import left to a new host-approved tool call.

Written by the indexing model from the issue text.

Assessment

Tech stack
csharp
Domain
frontend, tooling
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Active
Clarity
Mostly clear
Newbie friendliness
65/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.