Skip to content

End to end#4

Merged
denibravo merged 23 commits intomirrulations:mainfrom
drekdrek:end-to-end
Apr 23, 2025
Merged

End to end#4
denibravo merged 23 commits intomirrulations:mainfrom
drekdrek:end-to-end

Conversation

@drekdrek
Copy link

@drekdrek drekdrek commented Apr 15, 2025

This is my current progress on the end-to-end system.

Currently the following works

  • Local cognito logins
  • The frontend of the website
  • The databases
  • The API for the website
  • The ETL process

@ZingBing
Copy link

I know this isn't important yet but for the ETL side as a reminder we could just redirect the sql and opensearch setup in the submodule using the global AWS_SAM_LOCAL variable

@CloudUki
Copy link

For the API, is there anything that we would have to do or is DB. Sorry if I am not understanding. :/

Choose a reason for hiding this comment

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

This self explanatory, but it looks good to me.

Copy link
Contributor

Choose a reason for hiding this comment

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

LGTM
Dancing_Duck

@drekdrek
Copy link
Author

For the API, is there anything that we would have to do or is DB. Sorry if I am not understanding. :/

the current issue with it is just how it chooses local vs prod credentials, its the db side of it

@CloudUki
Copy link

I see thank you for clarification :) .

@drekdrek
Copy link
Author

this is currently dependent on other PRs being merged downstream.

@drekdrek drekdrek marked this pull request as ready for review April 23, 2025 17:25
Copy link
Contributor

@elon-shmusk elon-shmusk left a comment

Choose a reason for hiding this comment

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

I went through it myself and was easily able to follow along and get it working. Great job on the readme!

Copy link
Contributor

@denibravo denibravo left a comment

Choose a reason for hiding this comment

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

Documentation was straight forward and easy to follow. Good job on this. Approved

@denibravo denibravo merged commit 238ee5f into mirrulations:main Apr 23, 2025
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.

5 participants