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

Unchecked conversion between data types in metafields #276

Open
ashmaroli opened this issue Jan 14, 2017 · 3 comments
Open

Unchecked conversion between data types in metafields #276

ashmaroli opened this issue Jan 14, 2017 · 3 comments
Labels

Comments

@ashmaroli
Copy link
Member

ashmaroli commented Jan 14, 2017

Description:

Tell us a bit about yourself:

  • Version of JekyllAdmin I'm using : 0.2.0
  • Version of Jekyll I'm using : 3.3.1
  • Operating System : Windows
  • Browser : Chrome

Steps to reproduce:

  1. Create a new document via Admin interface
  2. Create a complex set of metadata:
    2.1 value-objects
    2.2 objects with lists
    2.3 lists with objects

I expected the following:

expected the generated Front Matter to be a well-formatted valid YAML data

But got the following, instead:

  • the key has whitespaces between words instead of an underscore '_'
  • the first item of nested list-items have 2 dashes - - next to them

though http://www.yamllint.com/ doesn't flag the resulting data as INVALID, this was unexpected.

An example set of Front Matter data generated by a novice:

title: Test
foo: bar
template: post
pro players:
  forwards:
  - Harvey
  - Webber
  defence:
  - Campbell
  - - Alvin
    - Dave
    - Substitute players:
      - Chad
      - Billy

Other details:

lint

@jekyllbot
Copy link

This issue has been automatically marked as stale because it has not been commented on for at least two months.

The resources of the Jekyll team are limited, and so we are asking for your help.

If this is a bug and you can still reproduce this error on the master branch, please reply with all of the information you have about it in order to keep the issue open.

If this is a feature request, please consider whether it can be accomplished in another way. If it cannot, please elaborate on why it is core to this project and why you feel more than 80% of users would find this beneficial.

This issue will automatically be closed in two months if no further activity occurs. Thank you for all your contributions.

@mertkahyaoglu
Copy link
Member

@ashmaroli is this still an issue?

@jekyllbot jekyllbot removed the stale label May 17, 2017
@ashmaroli
Copy link
Member Author

The following was created from the master branch:

---
title: foo
all: true
page_only: true
foo:
- bar
- - lorem
  - ipsum
  - dolor: sit
    sit: amet
    blah:
    - blah
---

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants