Skip to content

🐛 Fixed cardWidth being lost on 2.0 imports#10068

Merged
kirrg001 merged 2 commits into
TryGhost:masterfrom
ErisDS:fix-koenig-import
Oct 26, 2018
Merged

🐛 Fixed cardWidth being lost on 2.0 imports#10068
kirrg001 merged 2 commits into
TryGhost:masterfrom
ErisDS:fix-koenig-import

Conversation

@ErisDS
Copy link
Copy Markdown
Member

@ErisDS ErisDS commented Oct 25, 2018

Came across this whilst building our html->to->mobiledoc migration tooling. I'm correctly setting cardWidth in the parser plugins, and it's correctly being output in the mobiledoc, but Ghost import is throwing the cardWidth away.

no issue

  • When importing Ghost 2.0 blogs into 2.0 blogs...
  • The Koenig image card would lose it's cardWidth setting,
  • because it'd be overridden by the imageStyle setting, which was null
  • The importer previous only kept the width if importing 1.0 blogs

no issue

- When importing Ghost 2.0 blogs into 2.0 blogs...
- The Koenig image card would lose it's cardWidth setting,
- because it'd be overridden by the imageStyle setting, which was null
- The importer previous _only_ kept the width if importing 1.0 blogs
@ErisDS ErisDS requested a review from kirrg001 October 25, 2018 20:41
@ErisDS ErisDS added bug [triage] something behaving unexpectedly klitzeklein labels Oct 25, 2018
Comment thread core/test/integration/data/importer/importers/data_spec.js Outdated
@kirrg001 kirrg001 merged commit 275d821 into TryGhost:master Oct 26, 2018
allouis added a commit that referenced this pull request Oct 30, 2018
* master:
  Version bump to 2.4.0
  Updated Ghost-Admin to 2.4.0
  Handled error from express-session middleware (#10084)
  Removed user_id constraint when upserting session (#10085)
  Added Node v10 Support (#10058)
  Removed logic for migration script (2.0/6-replace-fixture-posts.js) (#10081)
  Fixed sanitization issue in subscribers
  Parsed nconf env values (#10077)
  🐛 Fixed missing filename when exporting subscribers csv
  🐛 Fixed pagination for subscribers
  🐛 Fixed cardWidth being lost on 2.0 imports (#10068)
  Corrected 'Content-Length' header by using Buffer.byteLength (#10055)
  Fixed mail api usage of the notifcations api
  Refactored request unit tests to return promises (#10045)
  Refactored spam prevention tests to use promises (#10036)
  Enforced unix line endings (#9871)
@ErisDS ErisDS deleted the fix-koenig-import branch August 27, 2019 14:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug [triage] something behaving unexpectedly

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants