[3.0.0-beta.34] Error with Payload config validation on field "admin.meta" about payload HOT 8 CLOSED

Hey @nvti, meta.icons is supposed to be an array. That way you can set multiple icons in the head of your document, i.e. light / dark mode. Looks like we might've overlooked your PR unfortunately. I really appreciate the contribution regardless. This change did go out in the last beta release, so I went ahead and opened #6759 to resolve this—let me know what you think.

Hi @jacobsfletch, thank you for your reply. My PR does only 1 thing: sync between MetaConfig type and its validation schema.
I think changing meta.icons to array is OK, your PR is great and I don't have any comment about that.

BTW, there is another small thing in my PR about OpenGraphConfig type (this type is changed, too). Please check in my PR (#6426). If it looks OK, please copy it into your PR. Thanks
packages/payload/src/config/types.ts

export const openGraphSchema = joi.object({
  description: joi.string(),
  images: joi.alternatives().try(ogImageObj, joi.array().items(ogImageObj)),
  siteName: joi.string(),
  title: joi.string(),
})

from payload.

Ahh, rip. Would you be willing to PR for this? Otherwise I will get to this as soon as I can.

from payload.

Merged. Thank you @nvti 🙌

from payload.

Hey @nvti, meta.icons is supposed to be an array. That way you can set multiple icons in the head of your document, i.e. light / dark mode. Looks like we might've overlooked your PR unfortunately. I really appreciate the contribution regardless. This change did go out in the last beta release, so I went ahead and opened #6759 to resolve this—let me know what you think.

from payload.

Got it 👍 that has been carried over here:

from payload.

@jacobsfletch you still forgot the siteName field 😄 It's OK, just a small thing. You can add this in another PR.
the new type of OpenGraphConfig:

export type OpenGraphConfig = {
    description?: string;
    images?: OGImageConfig | OGImageConfig[];
    siteName?: string;
    title?: string;
};

from payload.

@jacobsfletch ok, let me open a new PR for this

from payload.

@jacobsfletch I created PR #6764 to address this

from payload.