Skip to content

Fix #90: Update parent to params.author in "Configure site author" #91

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

irusukan
Copy link
Contributor

@irusukan irusukan commented May 5, 2025

Overview

Fixes #90: Updates the parent value from author to params.author in the menu list generation process from configOptions, executed when selecting " Configure site author".

Issue Details

Problem:

When the "Configure site author" command is executed in the CLI, no menu list is displayed in the terminal.

Reproduction Steps:

  1. Launch "blowfish-tools" (v1.10.0).
  2. Select "Setup anew website with Blowfish" and enter an arbitrary site name.
  3. Select "Configure site author".

Cause:

The command generates a menu list from the configOptions object using data where parent is set to author. The list is empty because the correct parent key is params.author.

Changes

Updated the parent value from author to params.author.

Testing

Manually verified the "Configure site author" command in Blowfish-Tools (v1.10.0) to ensure the menu list displays correctly.
confirm


Thanks for reviewing!

irusukan added a commit to irusukan/blowfish-tools that referenced this pull request May 6, 2025
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.

"Configure site author" fails to display the menu list
1 participant