Skip to content
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

🐛 Fixed cardWidth being lost on 2.0 imports #10068

Merged
merged 2 commits into from
Oct 26, 2018

Conversation

ErisDS
Copy link
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
@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