Skip to content
This repository has been archived by the owner on Oct 2, 2018. It is now read-only.

Backend Deployed to AWS #156

Open
wants to merge 9 commits into
base: develop
Choose a base branch
from
Open

Backend Deployed to AWS #156

wants to merge 9 commits into from

Conversation

kenan-rhoton
Copy link
Owner

Description

These changes are intended to smoothly integrate the backend into a AWS structure

Issues Resolved

This relates to #120, solving the Backend aspect of it.

Details

So there are quite a few changes:

  1. Eliminated compojure to use an AWS Lambda-Compliant method, where -handleRequest is the entrypoint.
  2. Switched from Redis to DynamoDB (ran into many problems using Amazon's Redis...)
  3. Configuration is now hard-coded since the Lambda function doesn't have access to the YAML

Hotspots

The entire API Handler has been changed. However, one of the side-effects is that it is almost entirely testable now (which means I should definitely add tests...), except for the -handleRequest itself.

It should still serve the same exact API, although right now I haven't figured out how to run it locally to test it (I have been doing executions using the REPL to test it locally so far...)

This PR should still be considered Work In Progress

@kenan-rhoton
Copy link
Owner Author

I think this PR needs to be blocked until frontend correctly matches with the current backend, maybe?

I'm afraid many frontend changes will have to be done to get everything working properly... Or maybe not, I'll think about it

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

Successfully merging this pull request may close these issues.

1 participant