Skip to content

Fix CheckValidationAttributes call within ReviewWizardPage not outputting all error messages#35

Closed
TrvsF wants to merge 8 commits intoFacepunch:masterfrom
TrvsF:master
Closed

Fix CheckValidationAttributes call within ReviewWizardPage not outputting all error messages#35
TrvsF wants to merge 8 commits intoFacepunch:masterfrom
TrvsF:master

Conversation

@TrvsF
Copy link
Copy Markdown
Contributor

@TrvsF TrvsF commented Nov 27, 2025

i was getting a silent error when trying to publish a project with a Title above 32 characters
this change makes any errors that occur when Project.Config contains invalid properties appear to the user

image

@TrvsF TrvsF marked this pull request as ready for review November 27, 2025 01:09
@Madmegsox1
Copy link
Copy Markdown

looks like a very useful pr, i think Ive gotten stuck here once or twice...

@handsomematt handsomematt added the triaged triaged pull-requests are replicated on the internal sbox repo label Jan 15, 2026
@sboxbot sboxbot added the accepted this pull request was accepted, hurrah! label Jan 15, 2026
@sboxbot
Copy link
Copy Markdown
Contributor

sboxbot commented Jan 15, 2026

This PR has been merged upstream.

@sboxbot sboxbot closed this Jan 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

accepted this pull request was accepted, hurrah! triaged triaged pull-requests are replicated on the internal sbox repo

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants