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

Tags screen refresh #1283

Merged
merged 25 commits into from Aug 27, 2019
Merged

Tags screen refresh #1283

merged 25 commits into from Aug 27, 2019

Conversation

rishabhgrg
Copy link
Contributor

@rishabhgrg rishabhgrg commented Jul 30, 2019

Updates tags list and tag detail screens

  • Adds new tag list component for list page following same design pattern as posts/page - gh-tags-list-item
  • Adds tab button/view for internal/public tags
  • Adds new tag detail screen for editing/creating tags
  • Creating new tags and editing existing now needs to be explicitly saved
  • Added slug generation based on name for new tag using utils

@rishabhgrg
Copy link
Contributor Author

Build fails currently as tests have not been updated to work with new tag routing/structure

app/routes/tags/index.js Outdated Show resolved Hide resolved
@rishabhgrg rishabhgrg force-pushed the tags-refresh branch 2 times, most recently from 26d2712 to 529a2a2 Compare August 21, 2019 07:41
@rishabhgrg rishabhgrg changed the title [WIP] Tags UI update Tags screen refresh Aug 21, 2019
@@ -38,7 +38,7 @@ let keyup = function (code, el) {
(el || document).dispatchEvent(event);
};

describe('Acceptance: Tags', function () {
describe.skip('Acceptance: Tags', function () {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The acceptance here covers the old behavior and are WIP for new behavior, so skipped them here and for the tag-settings component and will be replaced with updated tests

@rishabhgrg rishabhgrg merged commit 166c8ff into TryGhost:master Aug 27, 2019
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