Skip to content
This repository has been archived by the owner on Nov 28, 2022. It is now read-only.

⚡️ Add per-post code injection fields to post settings menu #811

Merged
merged 2 commits into from
Aug 2, 2017

Conversation

kevinansfield
Copy link
Collaborator

@kevinansfield kevinansfield commented Aug 1, 2017

refs tbd, requires tbd

  • add "Code Injection" pane to the PSM
  • implement codeinjectionHead and codeinjectionFoot attributes on Post model and save values from PSM

TODO:

  • use CodeMirror for injection fields
  • make fields taller?

@kevinansfield kevinansfield changed the title ⚡️ Add per-post code injection fields to post settings menu [WIP] ⚡️ Add per-post code injection fields to post settings menu Aug 1, 2017
@coveralls
Copy link

Coverage Status

Coverage increased (+0.3%) to 73.344% when pulling 9347108 on kevinansfield:per-post-code-injection into fb316a9 on TryGhost:master.

no issue

The PSM was the only place where the `gh-tabs-manager`, `gh-tab`, and `gh-tab-pane` components were being used. These were very old components and did not work well with newer Ember versions and best practices.

- 🔥 remove `gh-tab*` components
- 💄 fix indents in `gh-post-settings-menu` template
- 🎨 add support for named subviews ready for additional PSM panes
@kirrg001 kirrg001 changed the title [WIP] ⚡️ Add per-post code injection fields to post settings menu ⚡️ Add per-post code injection fields to post settings menu Aug 2, 2017
@coveralls
Copy link

Coverage Status

Coverage increased (+0.3%) to 73.304% when pulling 3c84074 on kevinansfield:per-post-code-injection into fb316a9 on TryGhost:master.

- add "Code Injection" pane to the PSM
- implement `codeinjectionHead` and `codeinjectionFoot`  attributes on `Post` model and save values from PSM
- use CodeMirror for the PSM code injection fields
@coveralls
Copy link

Coverage Status

Coverage increased (+0.3%) to 73.295% when pulling 264b079 on kevinansfield:per-post-code-injection into fb316a9 on TryGhost:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.3%) to 73.295% when pulling 264b079 on kevinansfield:per-post-code-injection into fb316a9 on TryGhost:master.

@aileen aileen merged commit 7c1f6e1 into TryGhost:master Aug 2, 2017
@aileen aileen mentioned this pull request Aug 2, 2017
@kevinansfield kevinansfield deleted the per-post-code-injection branch June 4, 2018 09:52
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
3 participants