Skip to content

fix #92 live logs #93

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

fix #92 live logs #93

wants to merge 1 commit into from

Conversation

oallain
Copy link
Member

@oallain oallain commented Jan 20, 2024

Q A
Bug fix? yes
New feature? no
BC breaks? no
Fixed issue #92
License MIT

@oallain oallain linked an issue Jan 20, 2024 that may be closed by this pull request
@oallain oallain closed this Sep 11, 2024
@oallain oallain reopened this Sep 11, 2024
@CLAassistant
Copy link

CLAassistant commented Sep 11, 2024

CLA assistant check
All committers have signed the CLA.

@timmess timmess force-pushed the fix/#92-live-logs branch 7 times, most recently from 92773d7 to e4e434b Compare September 12, 2024 12:36
@oallain oallain force-pushed the fix/#92-live-logs branch from e4e434b to 4af25c3 Compare March 22, 2025 17:47
@oallain oallain requested a review from maxperei March 22, 2025 17:52
@oallain oallain force-pushed the fix/#92-live-logs branch from 4af25c3 to 436b012 Compare March 22, 2025 20:48
@@ -68,6 +68,10 @@
cp -R vendor/synolia/sylius-scheduler-command-plugin/install/Application/config/packages/* config/packages/
cp -R vendor/synolia/sylius-scheduler-command-plugin/install/Application/config/routes/* config/routes/

6. Installing assets (JS and CSS) fot the plugin
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
6. Installing assets (JS and CSS) fot the plugin
6. Installing assets (JS and CSS) for the plugin

@@ -69,6 +69,7 @@ install-sylius:
${CONSOLE} sylius:fixtures:load default -n
${YARN} install
${YARN} build
${CONSOLE} assets:install -n
Copy link
Member

Choose a reason for hiding this comment

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

i'm wondering if composer req does not already play this command at auto-scripts 🤔

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.

[Bug] "Live tail" not working due to missing js
3 participants