Powering HackIllinois #50
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ASankaran
requested review from
justinyangusa,
redsn0w422,
apa-123,
smg155 and
a team
|
I think most of the post is really solid. The individual section on each service may not be necessary for this post. If there were some services that had interesting challenges then we may be able to spin it out into another post. |
|
Great start, @ASankaran! Based on our conversation yesterday, I’ll be looking for the next revision to be shorter with additional details on the microservice architecture decision. Descriptions for each microservice can be moved into documentation for the API. |
redsn0w422
requested changes
Aug 14, 2018
| There are a certainly more features which would be very useful for both staff and hackers at HackIllinois, but these are the core requirements we will start off with. Once completed, we can explore more advanced functionality. | ||
|
|
||
| ### Implementation Strategy | ||
| As mentioned at the begining of the post, there have been a few iterations of the API over the years. The purpose and requirements of the API over the years has remainds relativley constant, however the implementation has greatly different from iteration to iteration. Each iteration of the API aims to innovate and take a new approach to solving problems which arose in the past. [Last year's API](https://github.com/HackIllinois/api-2018) was implemented as a monolithic application using nodejs and MySQL for the database. It worked and cewrtainl met the requirements defined for HackIllinois 2018, however some issue we ran into during the development was the lack of compile time type safety and annoyance of building a new database migration for any minor change to the defined scheme. This year we aimed to reimplement the API from scratch and resolve these issues. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Closes #30.
This post is about the HackIllinois API and some the decisions that were made while designing and implementing it.
Pre-Approval
featuredtag has been added to this postfeaturedtag has been removed from other postsPre-Deploy