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

Storefront 001 #21

Merged
merged 2 commits into from
Jun 19, 2023
Merged

Storefront 001 #21

merged 2 commits into from
Jun 19, 2023

Conversation

jschluchter
Copy link
Collaborator

Purpose

Adding server side processing to store-front to allow proper function in AKS (k8s)

  • ...

Does this introduce a breaking change?

[X] Yes
[ ] No

Pull Request Type

What kind of change does this Pull Request introduce?

[ ] Bugfix
[X] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

How to Test

  • Get the code
git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
docker compose build --no-cache
docker compose up
  • Test the code
    Get everything up and running via docker compose, get to storefront (either localhost:8080 or 0.0.0.0:8080)
    Ensure products load
    Ensure items added to cart
    Ensure order is processed

What to Check

Same as what to test

  • ...

Other Information

@chzbrgr71
Copy link
Collaborator

Tested this in AKS and the store-front is working. Well done.

I will merge.

@chzbrgr71 chzbrgr71 merged commit f56c9b5 into main Jun 19, 2023
6 checks passed
@pauldotyu pauldotyu deleted the storefront-001 branch July 19, 2023 20:38
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