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

Document workaround for pulling with table prefixes #171

Merged
merged 5 commits into from
Oct 5, 2023

Conversation

xaqrox
Copy link
Contributor

@xaqrox xaqrox commented Apr 24, 2023

Quick tip for a Pantheon edge case. I'm not sure if I have the warning syntax right, or if it can handle a code block inside...

@netlify
Copy link

netlify bot commented Apr 24, 2023

Deploy Preview for lando-pantheon ready!

Name Link
🔨 Latest commit d6fcc82
🔍 Latest deploy log https://app.netlify.com/sites/lando-pantheon/deploys/651f0e50f754d50008aaedb0
😎 Deploy Preview https://deploy-preview-171--lando-pantheon.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@reynoldsalec
Copy link
Member

@xaqrox do you happen to have the verbatim error message handy? Might be good to include that in the warning message, particularly if Google picks it up for those looking for the solution :)

@xaqrox
Copy link
Contributor Author

xaqrox commented Jun 9, 2023

@reynoldsalec just saw your comment! I did that, and then thought, "what if the error message was more helpful" so I did that too. We could even take it a step further and make the message something like:

lando_red "Couldn't find expected tables ($PULL_DB_CHECK_TABLE)"
lando_pink "If that's the wrong table to expect, set LANDO_DB_USER_TABLE=\"your_table_name_here\" in the container's environment"
exit 1

@xaqrox
Copy link
Contributor Author

xaqrox commented Jun 9, 2023

Of course, I guess that makes it no-longer-googleable for folks with the original error...

@reynoldsalec reynoldsalec merged commit 049f79c into lando:main Oct 5, 2023
8 of 26 checks passed
@reynoldsalec
Copy link
Member

...finally moving this along, thanks @xaqrox!

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