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

Tom Magyar assessment (Spring Boot, Jackson, Vue.js, Clusterize.js, Gradle, App Engine Standard) #7

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

Conversation

aTomSoftware
Copy link

I've finished the backend and deployed it on GAE here: http://hstonedemo.appspot.com/
I'm still working on the frontend, but some feedback is already appreciated 😉

@aTomSoftware aTomSoftware changed the title This is where I got before Monday noon Tom Magyar assessment (Spring Boot, Jackson, Vue.js, Clusterize.js, Gradle, App Engine Standard) Dec 7, 2017
@aTomSoftware
Copy link
Author

UI added, and can be tested here: https://hstonedemo.appspot.com/ui.html
Although many image URLs in the JSON are broken, and image download is slow from their hosting providers, it works quite stable on Windows:

  • Latest Google Chrome
  • Latest Firefox
  • Latest Edge

but I haven't tested extensively. There's always room for improvement, but this is likely to be my final version before the interview.

corrado4eyes added a commit to corrado4eyes/HeartstoneAssessment that referenced this pull request Jan 16, 2020
corrado4eyes added a commit to corrado4eyes/HeartstoneAssessment that referenced this pull request Jan 16, 2020
corrado4eyes added a commit to corrado4eyes/HeartstoneAssessment that referenced this pull request Jan 16, 2020
corrado4eyes added a commit to corrado4eyes/HeartstoneAssessment that referenced this pull request Jan 16, 2020
corrado4eyes added a commit to corrado4eyes/HeartstoneAssessment that referenced this pull request Jan 16, 2020
corrado4eyes added a commit to corrado4eyes/HeartstoneAssessment that referenced this pull request Jan 16, 2020
corrado4eyes added a commit to corrado4eyes/HeartstoneAssessment that referenced this pull request Jan 16, 2020
corrado4eyes added a commit to corrado4eyes/HeartstoneAssessment that referenced this pull request Jan 16, 2020
corrado4eyes added a commit to corrado4eyes/HeartstoneAssessment that referenced this pull request Jan 16, 2020
corrado4eyes added a commit to corrado4eyes/HeartstoneAssessment that referenced this pull request Jan 16, 2020
corrado4eyes added a commit to corrado4eyes/HeartstoneAssessment that referenced this pull request Jan 16, 2020
corrado4eyes added a commit to corrado4eyes/HeartstoneAssessment that referenced this pull request Jan 16, 2020
corrado4eyes added a commit to corrado4eyes/HeartstoneAssessment that referenced this pull request Jan 16, 2020
corrado4eyes added a commit to corrado4eyes/HeartstoneAssessment that referenced this pull request Jan 16, 2020
corrado4eyes added a commit to corrado4eyes/HeartstoneAssessment that referenced this pull request Jan 16, 2020
corrado4eyes added a commit to corrado4eyes/HeartstoneAssessment that referenced this pull request Jan 16, 2020
corrado4eyes added a commit to corrado4eyes/HeartstoneAssessment that referenced this pull request Jan 16, 2020
corrado4eyes added a commit to corrado4eyes/HeartstoneAssessment that referenced this pull request Jan 16, 2020
corrado4eyes added a commit to corrado4eyes/HeartstoneAssessment that referenced this pull request Jan 16, 2020
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.

1 participant