Skip to content

Fix case sensitivity in choose_alliance command - #5569

Closed
saikatdas0790 wants to merge 1 commit into
clockworklabs:masterfrom
saikatdas0790:patch-2
Closed

Fix case sensitivity in choose_alliance command#5569
saikatdas0790 wants to merge 1 commit into
clockworklabs:masterfrom
saikatdas0790:patch-2

Conversation

@saikatdas0790

Copy link
Copy Markdown

Corrected the casing of 'Good' to 'good' in the command example.

Description of Changes

Fixed the casing otherwise the command would not work and show this error:

Error: Invalid arguments provided for reducer `choose_alliance` for database `yral-database-spacetime-4lbo7` resolving to identity `c2000bc0eb62dd948f318073cba2dd434656d08046b0e80c32c459bdb79b2877`.

The reducer has the following signature:
        choose_alliance(alliance: enum { good: {}, neutral: {}, evil: {} })

Caused by:
    0: Response text: invalid arguments for reducer choose_alliance: [0].Good: invalid arguments for function choose_alliance: [0].Good: [0].Good: unknown variant `Good`, expected , `good`one of `neutral`, `evil` at line 1 column 9
    1: HTTP status client error (400 Bad Request) for url (http://127.0.0.1:3000/v1/database/c2000bc0eb62dd948f318073cba2dd434656d08046b0e80c32c459bdb79b2877/call/choose_alliance)

Expected complexity level and risk

1

Testing

Tested locally

Signed-off-by: Saikat Das <saikatdas0790@gmail.com>
@bfops

bfops commented Jul 28, 2026

Copy link
Copy Markdown
Collaborator

Thank you for catching this! This prompted us to do a more holistic review of similar issues, so I'm going to close this PR in favor of #5604.

@bfops bfops closed this Jul 28, 2026
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.

2 participants