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

Split up website and API into microservices #110

Merged
merged 9 commits into from
Jan 27, 2024

Conversation

gchristov
Copy link
Owner

@gchristov gchristov commented Jan 24, 2024

What does this pull request change?

This PR:

  • Splits up the website and API into two microservices, using nginx as a reverse proxy to map /api to the API service to maintain the existing endpoints and not require Slack app update
  • Refactors some Gradle plugins to better support microservices
  • Tidies up the project structure to better support microservices
  • Switches to Docker Compose for running the project locally

How is this change tested?

Manually and with existing tests.


Writing Kotlin Multiplatform tests

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

@gchristov gchristov force-pushed the tidy-up-project-structure branch 2 times, most recently from c6bb804 to fc3ef36 Compare January 26, 2024 07:51
@gchristov gchristov changed the title Restructure project to better support microservice architecture Split up website and API into microservices Jan 26, 2024

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

@gchristov gchristov marked this pull request as ready for review January 27, 2024 08:04
Copy link

pulumi bot commented Jan 27, 2024

🍹 The Update (preview) for gchristov/thecodinglove/prod was successful.

Resource Changes

    Name                         Type                            Operation
~   landing-page-service-binary  docker:index/image:Image        update
~   app-service                  gcp:cloudrunv2/service:Service  update
~   landing-page-service         gcp:cloudrunv2/service:Service  update
~   app-service-binary           docker:index/image:Image        update

@gchristov gchristov merged commit 31ca817 into master Jan 27, 2024
4 checks passed
@gchristov gchristov deleted the tidy-up-project-structure branch January 27, 2024 08:05
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.

None yet

1 participant