Skip to content
This repository was archived by the owner on Mar 20, 2025. It is now read-only.

fix: deployment issue with stripe (closes #399) #444

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

sinedied
Copy link
Contributor

Had to update a few packages, and the stripe api version.

To make sure this won't happen again, I renamed the build:ts script to build so it the stripe package will be built with the others on the CI.

anfibiacreativa
anfibiacreativa previously approved these changes Mar 2, 2024
@manekinekko
Copy link
Collaborator

FYI, a fix has alrady been merged to develop via https://github.com/Azure-Samples/contoso-real-estate/pull/416/files

@sinedied sinedied changed the base branch from main to develop March 5, 2024 09:42
@sinedied
Copy link
Contributor Author

sinedied commented Mar 5, 2024

@manekinekko I've rebased the branch from develop, should be good now

manekinekko
manekinekko previously approved these changes Mar 6, 2024
Base automatically changed from develop to main March 6, 2024 17:44
@manekinekko manekinekko dismissed their stale review March 6, 2024 17:44

The base branch was changed.

manekinekko
manekinekko previously approved these changes Mar 8, 2024
@anfibiacreativa
Copy link
Member

@sinedied can you please resolve conflicts and merge. Thanks!

@manekinekko
Copy link
Collaborator

@sinedied I think you've accidentally pushed a chat-sample submodule. Could you please delete it? :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants