Skip to content

Conversation

kbond
Copy link
Member

@kbond kbond commented Jun 22, 2023

Q A
Bug fix? no
New feature? no
Tickets n/a
License MIT

I ran into a bit of trouble running the test suite locally so I made some changes:

  • installed foundry to help with resetting/seeding the test db
  • fixed deprecations
  • updated deps

I also added a readme to detail the process for running the site locally.

@kbond
Copy link
Member Author

kbond commented Jun 22, 2023

When using the turbo stream chat demo, I get the following error when posting a message in chat:

Failed-to-send-an-update-500-Internal-Server-Error-

Must be to do with the MERCURE_URL variables in .env but not sure how to fix (or what to set it to locally).

@weaverryan
Copy link
Member

About the Mercure stuff, hmm. Iirc, using Docker + the symfony binary should set those automatically... though I'm not seeing that locally right now. I talk very specifically in a tutorial about this, but I can't seem to see the MERCURE_ env vars locally...

@kbond
Copy link
Member Author

kbond commented Jun 22, 2023

I should note that when I reload the page, the chat box contains my message - so appears some kind of side-effect is failing.

@weaverryan
Copy link
Member

@kbond I think I fixed it over on #965 - looks like wrong flag, or an outdated flag (I'm sure this worked at some point - I've used this locally).

weaverryan added a commit that referenced this pull request Jun 23, 2023
This PR was merged into the 2.x branch.

Discussion
----------

[Site] Fixing docker config for mercure

| Q             | A
| ------------- | ---
| Bug fix?      | yes (to the site)
| New feature?  | no
| Tickets       | None
| License       | MIT

The flag was wrong, so mercure wouldn't start property. Helps with #963

Cheers!

Commits
-------

3f4712e [Site] Fixing docker config for mercure
@weaverryan
Copy link
Member

Thanks Kevin!

@weaverryan weaverryan merged commit 46d885d into symfony:2.x Jun 23, 2023
@kbond kbond deleted the site-contrib-help branch June 23, 2023 00:07
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.

2 participants