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

Conversation

@tylers-username
Copy link
Collaborator

@tylers-username tylers-username commented Sep 24, 2025

Our demo presents itself as a multiapp Upsun project example. This restores the app back to it's originally designed multiapp structure as it is currently a frontend app mocking multiapp functionality.

No changes were made to the code other than the revert. Security fixes applied in addition to reverting to previous commit.

@github-actions
Copy link

Hello, and thank you for contributing to the Upsun demo project!

Setting up tests

Until all tests pass, a pull request will not be merged.

Build tests for combinations of Python and Node.js will happen automatically, and must pass in order to merge.
Another test will actually runthrough all steps of the demo on a new project, but requires a repository admin to add the runthrough label to the pull request.

Failing tests

If your PR is failing, see the Running tests section of the README for troubleshooting details.

Reviewing these changes locally:

  1. Clone this repo

    git clone -b revert/retore-multiapp-demo git@github.com:platformsh/demo-project.git demo-test-revert/retore-multiapp-demo && cd demo-test-revert/retore-multiapp-demo
  2. Create a project on Upsun:

    upsun project:create --title "Demo Project test (pr-424/merge)" --default-branch revert/retore-multiapp-demo

    This command will set the local repository remote automatically, but you will need to specify the organization and region.

  3. Push to Upsun:

    upsun push

Copy link
Collaborator Author

@tylers-username tylers-username left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed:

  • Revert commit
  • Dependency security warnings
  • Upsun demo flow Upsun CLI command changes to ensure passing tests

This was referenced Sep 24, 2025
@flovntp flovntp disabled auto-merge September 30, 2025 14:36
@flovntp flovntp merged commit 4cf7a56 into main Sep 30, 2025
13 checks passed
@flovntp flovntp deleted the revert/retore-multiapp-demo branch September 30, 2025 14:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants