3.0: moogBundle is not a valid top level property

To Reproduce

Step by step instructions to reproduce the behavior:

1. Add moogBundle property to any A3 module
2. Look at the CLI output (app crashed) with message something along the lines moogBundle is not a valid top level property

Expected behavior

Accept moogBundle property as a valid one

Describe the bug

moogBundle is required for bundled modules (such as Apostrophe Blog). There is a trace in https://github.com/apostrophecms/apostrophe/blob/3.0.0-alpha.3/index.js#L184 of the functionality and indeed mogBundle is required to be a top level property, but it seems the module validator is not accepting this prop.

Details

Version of Node.js: any

Server Operating System: any

Apostrophe CMS version 3.0.0-alpha.3

[boutell]

Miro, would you mind submitting a PR that adds this change and also renames the key to just bundle? Thanks!

On Mon, Jan 18, 2021 at 2:00 AM Miro Yovchev [e-mail] wrote:

Found it. For those interested in ‘monkey-patching’ (just override in your node modules, on every node_modules update) this while waiting for the next alpha/beta:

1. open moog.js https://github.com/apostrophecms/apostrophe/blob/3.0.0-alpha.3/lib/moog.js#L137 inside your node_modules/apostrophe folder
2. add ‘moogBundle’ to the validKeys` array

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/apostrophecms/apostrophe/issues/2686#issuecomment-762030255, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAH27I3YEKCPMFJBVGLIBLS2PMA3ANCNFSM4WDXOEOA .

–

THOMAS BOUTELL | CHIEF TECHNOLOGY OFFICER APOSTROPHECMS | apostrophecms.com | he/him/his

[myovchev]

Found it. For those interested in ‘monkey-patching’ (just override in your node modules, on every node_modules update) this while waiting for the next alpha/beta:

1. open moog.js inside your node_modules/apostrophe folder
2. add 'moogBundle' to the validKeys` array