chore: Attempting to merge Master in to (Theseus) Replatforming

[grmartin]

No description provided.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

❗ No coverage uploaded for pull request base (2u/replatforming@a5c446b). Click here to learn what that means.

Additional details and impacted files

@@                 Coverage Diff                 @@
##             2u/replatforming     #364   +/-   ##
===================================================
  Coverage                    ?   63.58%           
===================================================
  Files                       ?       34           
  Lines                       ?      335           
  Branches                    ?       74           
===================================================
  Hits                        ?      213           
  Misses                      ?      115           
  Partials                    ?        7           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[julianajlk]

LGTM, might be helpful to add which commits from master you're merging for future reference!

[pshiu]

It was really hard to see this given how much time has passed, but I think the only change to master from 2u/project-theseus and 2u/replatforming is just commit f545a5e to add the 2u/project-theseus branch to CI:

frontend-app-ecommerce % git log --oneline origin/master..origin/2u/replatforming 
a5c446b (origin/2u/replatforming, origin/2u/project-theseus) chore: Syncing 2u/project-theseus with master [2023-08-29] (#327)
f545a5e build: add 2u/project-theseus branch to run ci (#265)

We don't need f545a5e in master, so we may just want to close this PR & delete both branches, working from master from here on out.

[pshiu]

Closing PR as no longer needed!