Skip to content

Conversation

@DarshilRij
Copy link

Given Details:


This PR has been generated to review the feature viz. Feature: Creating Full stack application with the simple task management feature implemented in the feature/task-management branch.

Proposed Solution:
In response to the given details, I modified the given application code to the desired code by creating necessary frontend and backend features.

  • For the frontend side, the components and its styles are created to display the information received from the backend. Moreover, for state management in the react app I have used recoil to easily navigate among the states and resulting to less and non redundant code.

  • For the backend side, I started creating the additional routes for the tasks by taking some reference from the given todos. The serverless.yml file was configured to add the routes to the local server in order to access it to the frontend. I have make the use of concurrently to run this full stack application viz. frontend and backend at the same time

That is how, I managed to implement the given task successfully. The Acceptance Criteria viz. to have the insertion/updation and retrieval has been met in this project

@grahamsnz grahamsnz requested a review from JaonL May 4, 2022 20:03
@JaonL
Copy link
Contributor

JaonL commented May 4, 2022

Hi @DarshilRij, the backend runs fine, but the frontend seems to be missing an entry file to start from. Perhaps there was a file on your local version that hasn't been added into the commit yet?

@DarshilRij
Copy link
Author

Hi @DarshilRij, the backend runs fine, but the frontend seems to be missing an entry file to start from. Perhaps there was a file on your local version that hasn't been added into the commit yet?

Hi @JaonL, Sorry for this inconvenience, I will look into this matter and will request a re-review.

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