Refactor ducks

[ashmaroli]

No description provided.

[mertkahyaoglu]

This is not just validating metadata but also mutating path. I don't think we should mix things here. In order to understand what this function does to path, we need to look into it. It seems not write to me.

[ashmaroli]

So, what would be the best way to proceed yet avoid repeating the following logic in every duck (that handles metadata)?

const validateMetadata = (metadata, directory) => {
  let { path, title } = metadata;
  let errors = [];

  if ((!path || `${path}/` === directory) && title) {
    path = getFilenameFromTitle(title);
  } else {
    errors = validatePage(metadata);
  }
  return { path, errors };
};

[mertkahyaoglu]

@ashmaroli Is this ready to go?