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

API Rename validator classes #129

Merged
merged 1 commit into from
Dec 10, 2024

Conversation

emteknetnz
Copy link
Member

@emteknetnz emteknetnz changed the title API Renameator classes valid API Rename validator classes Dec 2, 2024
@emteknetnz emteknetnz changed the base branch from 5 to 5.0 December 4, 2024 00:43
@emteknetnz emteknetnz marked this pull request as ready for review December 9, 2024 05:05
Copy link
Member

@GuySartorelli GuySartorelli left a comment

Choose a reason for hiding this comment

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

LGTM will merge when all PRs are good to go

@GuySartorelli
Copy link
Member

CI failure shows

PHP Fatal error: Uncaught Error: Class "SilverStripe\Forms\RequiredFields" not found in /home/runner/work/silverstripe-spamprotection/silverstripe-spamprotection/vendor/silverstripe/userforms/code/Extension/UserFormValidator.php:11

The userforms PR did update that reference, so I'm not sure why this is trying to pull in an older version of userforms? Can you please investigate? It seems like a legitimate problem, but I guess could just be CI composer cache or something.

@emteknetnz
Copy link
Member Author

emteknetnz commented Dec 10, 2024

I've updated the require-dev dep of userforms from ^7 to 7.x-dev, because it is currently just installing 7.0.0-alpha1. Alternative would be to manually release 7.0.0-alpha2.

@GuySartorelli GuySartorelli merged commit e805958 into silverstripe:5.0 Dec 10, 2024
8 checks passed
@GuySartorelli GuySartorelli deleted the pulls/5/valid branch December 10, 2024 20:41
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