Problem/Motivation

Part of #3582246: [meta] CLI in Core community initiative. Referenced from theCLI in Core Initiative spreadsheet.

Steps to reproduce

Working on a Drupal Core CLI Command issue > Test Commands

Update system.settings page.front to try valid and invalid values when running the CLI command.

Proposed resolution

To be determined.

Remaining tasks

User interface changes

To be determined.

Introduced terminology

To be determined.

API changes

To be determined.

Data model changes

To be determined.

Release notes snippet

To be determined.

Issue fork drupal-3594338

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

mradcliffe created an issue. See original summary.

mradcliffe’s picture

mradcliffe
he/him
Primary language English
commented
Issue summary: View changes
Parent issue: #3582246: [meta] CLI in Core community initiative » #3606055: [meta] CLI entry point roadmap to stable

Updated the parent to the "stable" meta, and added manual testing instructions.

bircher’s picture

bircher
Location 🇨🇿
commented
Component: config.module » configuration system
Assigned: Unassigned » bircher

bircher opened merge request !17203

bircher’s picture

bircher
Location 🇨🇿
commented
Status: Active » Needs review
Related issues: +#3594337: Add a command to export configuration

I think this is ready for review.
Please also consider the sibling issue #3594337: Add a command to export configuration

larowlan’s picture

larowlan
Location 🇦🇺🏝.au GMT+10
commented

Left a comment in the review about re-use of renderTable between import and export commands

moshe weitzman made their first commit to this issue’s fork.

moshe weitzman’s picture

moshe weitzman commented
Status: Needs review » Reviewed & tested by the community

I re-reviewed the code and manually tested. RTBC.

  • larowlan committed 0a04f467 on main 
    feat: #3594338 Add a command to import configuration

By: mradcliffe
By...
larowlan’s picture

larowlan
Location 🇦🇺🏝.au GMT+10
commented
Status: Reviewed & tested by the community » Fixed

Committed and pushed 0a04f467c34 to main. Thanks!

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.