-
Notifications
You must be signed in to change notification settings - Fork 7
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
Add items to feast-collection palatte #1702
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Couple of grammar fixes but otherwise looks fine
@@ -81,6 +81,21 @@ export const chefPalettes: PaletteOption[] = [ | |||
]; | |||
|
|||
export const feastCollectionPalettes: PaletteOption[] = [ | |||
{ | |||
id: 'apples_pears', | |||
name: 'Apples pears', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Apples & Pears
maybe?
@@ -231,6 +246,21 @@ export const feastCollectionPalettes: PaletteOption[] = [ | |||
}, | |||
imageURL: 'https://uploads.guim.co.uk/2024/10/15/Cheeses.png', | |||
}, | |||
{ | |||
id: 'coffee_donuts', | |||
name: 'Coffee donuts', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also, maybe Coffee & Donuts
?
id: 'rice', | ||
name: 'Rice', | ||
id: 'potatos', | ||
name: 'Potatos', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Apparently should be Potatoes
(I had to look this up! 😁 https://www.dictionary.com/browse/potato)
What's changed?
Recent request to add items to Palatte for feast collection branding.
refernce: https://trello.com/c/ysz5pJHN/3340-more-to-feast-collection-branding
Checklist
General
Client