search

open-sauced / pizza-verse

A repository for all things pizza.
MIT License
38 stars 55 forks source link

docs: add information about Griot #100

Closed CBID2 closed 2 months ago

CBID2 commented 2 months ago

Description

I noticed that there was no category for Caribbean food, so I created one and added information about Griot .

What type of PR is this? (check all applicable)

Related Tickets & Documents

Related to #86

Added to documentation?

[optional] What GIF best describes this PR or how it makes you feel?

CBID2 commented 2 months ago

Hey @CBID2,

Thank you for this addition! ✨

As I'm not too familiar in which continent Caribbean is located, I googled it. Most resources — I link to some of them below for your reference — say that it's part of North America.

That said, can you please list your addition under North America?

Thanks. 🙂

Made the change @adiati98! 😊 You are correct about the Caribbean being apart of North America. Sometimes I forget that. I guess because in the U.S., the Caribbean is not really connected to America. Lol 😆

CBID2 commented 2 months ago

Hey @adiati98. I made the changes

CBID2 commented 2 months ago

@CBID2 there are things to fix here.

Also, can you update your branch with the main branch? We've merged a PR that added Indian Samosa, but I don't see it here.

Thanks! ✨

Hey @adiati98. I made the changes.

CBID2 commented 2 months ago

Hey @CBID2,

Can you please ask for review once you've addressed all feedback? I see you quoted my comment, but you haven't addressed something there.

Thanks.

Hey @adiati98. I reread your previous feedback. The information for the Samosa is on line 61. For some odd reason, GitHub won’t let me link it here.

adiati98 commented 2 months ago

@CBID2, have you tried to pull it locally?

CBID2 commented 2 months ago

@CBID2, have you tried to pull it locally?

Yes @adiati98

CBID2 commented 2 months ago

Yay! Now everybody gets to enjoy the awesomeness of Haitian food @adiati98! 😊