Problem/Motivation

When installing the module, multiple unit types and APT content are included. As a result, we need to remove them if they are not relevant to our use case. Additionally, the unit blocks the import of deletion configurations, meaning the APT type must be removed before importing.

Proposed resolution

It might be wise to place these unit types in an example module instead.

Issue fork bat-3516437

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

ilyna created an issue. See original summary.

ilyna opened merge request !24

afagioli’s picture

afagioli
Location Rome
commented
Status: Active » Postponed (maintainer needs more info)

Makes sense to me.
Happy to give a few time to assist who's is gonna take care of this.

ilyna’s picture

ilyna commented

Hi @afagioli,

Thank you for your enthusiasm. I submitted a merge request here — could you let me know what else you’re expecting from me? What are your thoughts?

afagioli’s picture

afagioli
Location Rome
commented

Thanks a lot!
Updating development release, where the this very MR can be applied

  • afagioli committed 9789694e on 11.1.x 
    updated devel waiting for #3516437  #3516437#comment-16061826
afagioli’s picture

afagioli
Location Rome
commented

Please create a new MR against the most recent development version

afagioli’s picture

afagioli
Location Rome
commented
Version: 11.1.0-beta1 » 11.0.x-dev
Status: Postponed (maintainer needs more info) » Needs work

  • afagioli committed e1b28e47 on 11.1.x 
    enable module bugfix  #3516437  #3516437#comment-16061826
ilyna’s picture

ilyna commented
Assigned: ilyna » Unassigned
afagioli’s picture

afagioli
Location Rome
commented
Status: Needs work » Needs review
afagioli’s picture

afagioli
Location Rome
commented
Assigned: Unassigned » afagioli
Status: Needs review » Needs work
afagioli’s picture

afagioli
Location Rome
commented
Version: 11.0.x-dev » 11.1.x-dev

  • afagioli committed c5bc5b53 on 11.0.x 
    Issue #3516437 by ilyna: Update README with bat_example warnings

  • afagioli committed e1b28e47 on 11.0.x 
    enable module bugfix  #3516437  #3516437#comment-16061826

  • afagioli committed 9789694e on 11.0.x 
    updated devel waiting for #3516437  #3516437#comment-16061826

  • afagioli committed 4652d8ef on 11.1.x 
    Issue #3516437 by ilyna: Update README with bat_example production...
afagioli’s picture

afagioli
Location Rome
commented
Status: Needs work » Needs review
afagioli’s picture

afagioli
Location Rome
commented

README updated with production warnings about bat_example module (commit xxxxxxx).

The code change (moving example content to bat_example) was already merged in e4687e6. This completes the remaining documentation work.

Production sites should not enable bat_example. Clean installations no longer include problematic demo content.

Thanks @ilyna!

afagioli’s picture

afagioli
Location Rome
commented
Assigned: afagioli » Unassigned

  • afagioli committed ee274755 on 11.1.x 
    Issue #3516437 by ilyna: Simplify README and add entities overview

-...