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

Fixes #36910 - Expect ERB in URL for test webhook #68

Merged

Conversation

ofedoren
Copy link
Member

@ofedoren ofedoren commented Nov 9, 2023

No description provided.

@adamruzicka
Copy link
Contributor

The text alignment looks weird. Does it need to be centered?
image

Ftr here's how it looks on failure
image

@pondrejk
Copy link

Just from looking at the screenshots, the error message is now much more useful, I agree with text alignment not to be center

@ofedoren ofedoren force-pushed the bug-36910-fix-test-webhook-with-erb branch from 38378ad to 1e0d256 Compare November 10, 2023 16:02
@ofedoren
Copy link
Member Author

Thanks, @adamruzicka and @pondrejk, fixed the alignment:
ScreenShot-1699632115106

@adamruzicka adamruzicka merged commit 7ed08ed into theforeman:master Nov 13, 2023
6 checks passed
@adamruzicka
Copy link
Contributor

Thank you @ofedoren & @pondrejk !

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