Feature/taxonomy

[mattnield]

Motivation

Fixes #1 by adding basic taxonomy support.

Each taxonomy group can be returned as a nested group of taxonomy terms. So running something like this:

query MyQuery {
  allKontentTaxonomyPersonas {
    nodes {
      terms {
        terms {
          codename
          name
        }
        name
        codename
      }
      system {
        codename
        name
      }
    }
  }
}

will return:

{
  "data": {
    "allKontentTaxonomyPersonas": {
      "nodes": [
        {
          "terms": [
            {
              "terms": [
                {
                  "codename": "barista",
                  "name": "Barista"
                },
                {
                  "codename": "cafe_owner",
                  "name": "Cafe owner"
                }
              ],
              "name": "Coffee expert",
              "codename": "coffee_expert"
            },
            {
              "terms": [
                {
                  "codename": "coffee_lover",
                  "name": "Coffee lover"
                },
                {
                  "codename": "coffee_blogger",
                  "name": "Coffee blogger"
                }
              ],
              "name": "Coffee enthusiast",
              "codename": "coffee_enthusiast"
            }
          ],
          "system": {
            "codename": "personas",
            "name": "Personas"
          }
        }
      ]
    }
  }
}

Checklist

• Code follows coding conventions held in this repo
• Automated tests have been added
• Tests are passing
• Docs have been updated (if applicable)
• Temporary settings (e.g. variables used during development and testing) have been reverted to defaults

How to test

None of the existing automated tests were relevant. All testing was done through use fo the GraphiQL explorer and building basic listings in Gatsby

[mattnield]

Looks like I have an issue in the test scripts since merging in the latest master branch. I'll go back and figure out how to exclude the taxonomy from these tests because languages, tracing, and resolvers are not relevant to the taxonomy support.

[NathanDCox]

I made a new pull request that implements this in a similar way. There are some formatting fixes, I added in tests in gatsby-node and added validation. Mine still doesn’t resolve the unnecessary tests in languages and resolvers failing the build though.

[mattnield]

@NathanDCox - looks like we're working on the same thing then.

[mattnield]

Hi @Simply007 , all is up to date now with passing tests and documentation. Let me know if you want to drop on a call to discuss.

[NathanDCox]

@mattnield - Looks like you got a working update. Looking forward to using this feature on an upcoming project.

[Simply007]

Hey @mattnield!

thanks for the work you have done!

I do love the way you adopted the "architecture" of the plugin. 👍👍👍
Documentation + tests are also included. 🥇

Just one note:

• I would probably make the taxonomy support configurable to allow the users to turn it off
  • I am still not decided whether opt-in or opt-out is better for default - what do you think?
  • I would love to do the same with type summit

We could definitely schedule a call and discuss is.

[mattnield]

Hi @Simply007,

Thanks 😄

Yes, let's have a call about your suggestion:

• I would probably make the taxonomy support configurable to allow the users to turn it off

If you're free tomorrow afternoon, that could work? We can speak on Slack 🤙

[Simply007]

Sorry @mattnield to keep you waiting, but there is an issue with schema definition. When we have implemented the schema definition, the empty linked element value became null from empty array (well-spoted @bwlng).

Problem description here: https://github.com/Simply007/gatsby-null-instead-of-array

[Simply007]

Hello @mattnield,

finally, we have released the v4.0.0, so I am ready to merge your pull request and release it as the v4.2 (we will skip 4.1 because it is currently used for gatsby preview development and it will be moved to v4.3)

I have left some rebranding tasks to finish (the snapshot update might need to be done after the changes).

Overall it is great! Thanks for your contribution!

[mattnield]

Thanks, @Simply007. These are in and passing now.

[Simply007]

THX!

[Simply007]

Hello @NathanDCox. I've just sent you an invitation to the Kentico GitHub organization. If you accept it, you'll be able to interact with other Kentico GitHub community members in out Global collaborators dicsussions as well as getting the fresh info about Kentico Github activity.