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

Assesment solution #4

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

vardansaini
Copy link

Hi, I am Vardan Saini, a fourth year Undergraduate student at the University of Alberta. While going through the list of ideas for Eclipse Foundation, I came across "User-friendly API and configuration for the Eclipse Vert.x HTTP Proxy". I find this idea interesting and would like to work on this. As part of the application process, I have solved the assessment questions in this PR. Kindly review!

Create a /joke endpoint to retrieve random jokes from icanhazdadjoke.com. Also,
add a test for the /joke endpoint.
Create a simple website page with a button for the user to retrieve a new joke
from the backend service and display it to the user.

Added CORS for backend because website and backend run on different ports.
Configure a reverse proxy for website and backend, update /joke endpoint
url in website page and remove CORS because it is no longer needed.
Copy link
Owner

@tsegismont tsegismont left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, thank you

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