Skip to content

Add a trigger-bsas command#56

Merged
ac000 merged 1 commit into
masterfrom
bsas
Jun 24, 2026
Merged

Add a trigger-bsas command#56
ac000 merged 1 commit into
masterfrom
bsas

Conversation

@ac000

@ac000 ac000 commented Jun 24, 2026

Copy link
Copy Markdown
Owner

This is to trigger a Business Source Adjustable Summary for the given tax year or the current tax year if not specified.

Copilot AI review requested due to automatic review settings June 24, 2026 00:30
Comment thread src/itsa.c Dismissed

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.

Pull request overview

Adds a new CLI command to trigger a Business Source Adjustable Summary (BSAS) by building a JSON payload (derived from the current tax year by default, or from explicit start/end dates) and submitting it to the MTD API.

Changes:

  • Adds trigger-bsas command wiring (usage text + dispatcher entry).
  • Implements interactive BSAS payload creation/display/edit/submit flow with a temp JSON file.
  • Documents the new command in README.rst.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 6 comments.

File Description
src/itsa.c Implements the trigger-bsas command, including JSON payload generation, temp-file handling, interactive submit/edit loop, and dispatcher registration.
README.rst Adds trigger-bsas to the supported commands list.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/itsa.c
Comment thread src/itsa.c
Comment thread src/itsa.c
Comment thread src/itsa.c
Comment thread src/itsa.c
Comment thread README.rst
This is to trigger a Business Source Adjustable Summary for the given
tax year or the current tax year if not specified.

Signed-off-by: Andrew Clayton <ac@sigsegv.uk>

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.

Pull request overview

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

Comment thread src/itsa.c
Comment thread src/itsa.c
Comment thread src/itsa.c
@ac000 ac000 merged commit a83395b into master Jun 24, 2026
10 checks passed
@ac000 ac000 deleted the bsas branch June 24, 2026 01:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants