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

Goody (DoorDash) API Integration - Gift Cards 🎁 #187

Open
ramiAbdou opened this issue May 1, 2024 · 16 comments · May be fixed by #460
Open

Goody (DoorDash) API Integration - Gift Cards 🎁 #187

ramiAbdou opened this issue May 1, 2024 · 16 comments · May be fixed by #460
Assignees
Labels
Feature 🚀 A new feature or improvement.

Comments

@ramiAbdou
Copy link
Member

ramiAbdou commented May 1, 2024

Description

We typically reward members who are in the Top 75 of the leaderboard of a Fam Friday session with an AlgoExpert license as well as a DoorDash gift card. To administer the DoorDash gift cards, we use a service called Goody.

We want to build an integration with the Goody API so that we can make sending these gifts as easy as a few clicks in our Admin Dashboard. Eventually, we can build a deeper integration with the Airmeet API and automatically grab the top 75 in the leaderboard, so sending gift cards would be a 1-click task! The latter part we'll save for the next iteration.

For now, we'll build a simple integration with the Goody Automation API, which allows us to send gifts to multiple people.

Acceptance Criteria

This issue should do the following:

  • Implement the ability to send Goody gifts, using their API (ie: POST /v1/order_batches).
    • You can assume that the product_id is read from an environment variable (ie: GOODY_DOORDASH_PRODUCT_ID).
  • In the Admin Dashboard -> Students page, add an option when clicking on the row's option to "Send Goody Gift". This should open a modal which contains a form asking for the "Gift", which for now should have "DoorDash Gift Card" as the only option.
  • Upon submitting the form above, this should send the gift to the member. This should also send an internal Slack notification.

Don't worry about testing this - just implement 90% and I'll jump in to help testing with a real Goody API key.

@ramiAbdou ramiAbdou changed the title Goody API Integration Goody API Integration - Gifts (DoorDash) 🎁 May 1, 2024
@ramiAbdou
Copy link
Member Author

@sajjad-source

@sajjad-source
Copy link
Contributor

ty! would love to work on this

@ramiAbdou ramiAbdou added Feature 🚀 A new feature or improvement. Help Wanted 🤲 labels May 1, 2024
@ramiAbdou ramiAbdou changed the title Goody API Integration - Gifts (DoorDash) 🎁 Goody (DoorDash) API Integration - Gift Cards 🎁 May 2, 2024
@tomas-salgado
Copy link
Collaborator

Hey @sajjad-source! Are you still working on this issue? No worries if not, just let me know so we can open it up for someone else!

@tomas-salgado
Copy link
Collaborator

Going to open up this issue!

@sajjad-source
Copy link
Contributor

Hey @tomas-salgado. Really sorry I just saw your messages and had completely forgot about this issue 😅. Have had a lot on my plate since school ended, but I hope to be able to contribute again when space opens up. Thanks for opening up the issue again

@nicholasg2001
Copy link
Contributor

If this issue is back up for grabs I'm down to work on it!

@tomas-salgado
Copy link
Collaborator

@nicholasg2001 You've got it!

@nicholasg2001
Copy link
Contributor

@ramiAbdou Still learning the codebase so I just have a clarifying question. For the goody API code, should I be creating a folder goody inside of packages/core/src/modules?

@tomas-salgado
Copy link
Collaborator

@nicholasg2001 just realized we missed this question, apologies. For now it should be fine to just have the folder goody in that location.

Let me know if you still plan on working on this issue!

@omar-elbaz
Copy link
Contributor

I'd be down to take it @nicholasg2001 if you're no longer working on it. LMK

@nicholasg2001
Copy link
Contributor

You can take it!

@omar-elbaz
Copy link
Contributor

@tomas-salgado Im working on it! assign me

@omar-elbaz
Copy link
Contributor

Got a little bit of help from @RiosNicholas can we add him as an assignee as well? (bro barely did anything ngl but lets show him some love)

@omar-elbaz omar-elbaz linked a pull request Aug 18, 2024 that will close this issue
3 tasks
@RiosNicholas
Copy link
Contributor

Got a little bit of help from @RiosNicholas can we add him as an assignee as well? (bro barely did anything ngl but lets show him some love)

chill, I cooked with the frontend @omar-elbaz

@RiosNicholas
Copy link
Contributor

I think you assigned the wrong Nicholas, lol @tomas-salgado

@tomas-salgado
Copy link
Collaborator

Ahh mb 😆

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature 🚀 A new feature or improvement.
Projects
Status: In Progress
Development

Successfully merging a pull request may close this issue.

6 participants