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

Endpoint to search for github user #388

Merged
merged 6 commits into from
Sep 3, 2021
Merged

Conversation

PabloWolf96
Copy link
Contributor

Before submitting your PR for review

  • Run npm run eslint to find errors in code syntax/format
  • Run npm run eslint:fix to fix all auto-fixable errors in source code and auto-format with prettier
  • Ensure you fix any linting errors displayed after running any of the above commands
  • Ensure you fix all merge conflicts before sending PR

What does this PR do?

Fixes #378

Summarise the main tasks handled in the pull request
endpoint /api/github/user/ return json object with github user's information

Description of Task to be completed?

Describe (in detail) what the task completed in the pull request does as per the relevant task

How should this be manually tested?

Give bullet point instructions on how to setup the code and test it manually (Assume no prior experience on JS or code)

Any background context you want to provide?

Any pertinent information that should be considered

What is the link to the issue on Github?

Issue #240

Questions:

If something is unclear or you want some questions to be addressed by your peers, mention them here

@PabloWolf96
Copy link
Contributor Author

@Davien21 @alvacoder please review

@claretnnamocha claretnnamocha merged commit 30aa47a into zurichat:dev Sep 3, 2021
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.

Create an endpoint to search for a github user
2 participants