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

Deploy backend 2 #1494

Closed
wants to merge 1 commit into from
Closed

Deploy backend 2 #1494

wants to merge 1 commit into from

Conversation

davidyuk
Copy link
Member

@davidyuk davidyuk commented May 13, 2023

I was going to do it in the same way as #1489 but seems that in this case container doesn't get replaced, I either see "404 page not found" message or the container deployed on this branch previously. If I build and run this container locally using docker-compose it works.
@mmpetarpeshev can you check that is the issue here?

@github-actions
Copy link

Deployed to https://pr-1494-aepp-base.stg.aepps.com

@davidyuk davidyuk force-pushed the backend-temp-setup branch 6 times, most recently from 0609b83 to 4b452c6 Compare May 14, 2023 07:15
@mmpetarpeshev mmpetarpeshev requested a review from dincho May 14, 2023 16:34
@mmpetarpeshev
Copy link
Contributor

I was going to do it in the same way as #1489 but seems that in this case container doesn't get replaced, I either see "404 page not found" message or the container deployed on this branch previously. If I build and run this container locally using docker-compose it works. @mmpetarpeshev can you check that is the issue here?

@dincho support this service. If he prefers I do it. Will take a look, but let's sync with him first.
The pipelines don't support draft PRs.The GitHub events are important here.
I see only PR sync pipelines , which means something was wrong with the github workflow.
We can sync over chat what and how you did it.

@davidyuk davidyuk marked this pull request as ready for review May 19, 2023 10:15
@davidyuk davidyuk force-pushed the backend-temp-setup branch from 4b452c6 to c25c640 Compare May 19, 2023 10:17
@dincho
Copy link
Member

dincho commented May 19, 2023

What's the purpose of the PR? Server side instead of static SPA ?

@davidyuk
Copy link
Member Author

davidyuk commented Jul 3, 2023

Nope, it is to send push notifications and sync mobile and desktop versions of base aepp

@davidyuk davidyuk closed this Jan 15, 2024
@davidyuk davidyuk deleted the backend-temp-setup branch January 15, 2024 14:41
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.

3 participants