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

Spotify API Artist Search #16

Open
wants to merge 17 commits into
base: main
Choose a base branch
from

Conversation

dmartinezgamboa
Copy link

@dmartinezgamboa dmartinezgamboa commented Feb 23, 2022

My goal was to practice authenticating and requesting information from an API (Spotify in this case) and more back-end focused implementations.

The local server instance authenticates with Spotify and receives bearer token. In browser, user queries an artist which gets sent as a GET request to local server. Local server then requests information using query params & bearer token to Spotify, which local server then sends back to client (data is printed to console).

The intention was to keep client secrets and tokens away from client. Client hits your server and the server makes API calls to Spotify. Client only receives information.

I had alot of fun coding this and I appreciate the prompt. Due to getting this submitted in a timely manner, I wish I could have focused on some more features such as running the app in a container & testing frameworks. Both of which I have implemented in a separate repository (if you would like to see more of my work):
https://github.com/dmartinezgamboa/roastme-bot

Features:

  • node.js (express)
  • async/await patterns
  • styling/javascript to support html
  • repository features code linting and enforcement (eslint & prettier)

Wishlist (If I had more time):

  • Run application in Docker container.
  • More styling and actual UI/UX for presentation of data.
  • Creation of routers/controllers modules to clean up server.js
  • Testing framework. For now the application consists of two Spotify API call which I don't believe I need to test. With more routers testing would make more sense.
  • Bearer token expires after one hour, would like to implement logic to check if expired & refresh token.

@SpencerKaiser
Copy link
Contributor

@dmartinezgamboa can you post your email here? I can't seem to find your resume within your repo.

@dmartinezgamboa
Copy link
Author

dmartinezgamboa commented Mar 7, 2022

Hi @SpencerKaiser I have attached my resume.
My email is:
[email protected]
Software_Resume-Cover_Combined_Martinez-Gamboa.pdf

@SpencerKaiser
Copy link
Contributor

Great, I'll follow up via email shortly!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants