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

[5.x] Support non-revisable fields #11252

Open
wants to merge 19 commits into
base: 5.x
Choose a base branch
from

Conversation

edalzell
Copy link
Contributor

@edalzell edalzell commented Dec 12, 2024

We have a feature were you can sponsor an orphan. The sponsorship happens in the background (through folks signing up online).

We also have revisions. The issue is that if we want to revert back to some previous orphan data, we might be restoring a version that doesn't have a sponsorship, which will cause problems.

So we'd like to set the sponsorship fields as non-revisable so they persist.

@edalzell edalzell changed the title Support non-revisable fields [5.x] Support non-revisable fields Dec 12, 2024
@edalzell edalzell marked this pull request as ready for review December 13, 2024 19:43
@edalzell edalzell requested a review from jasonvarga December 13, 2024 19:43
@edalzell
Copy link
Contributor Author

I didn't see any tests for EntriesController nor TermsController, and honestly I was afraid to start them cuz I bet they need lots of setup.

Happy to start one, with some guidance

@jasonvarga
Copy link
Member

Those are in Tests\Feature\Entries\StoreEntryTest, UpdateEntry, EditEntryTest, etc.

@edalzell edalzell marked this pull request as draft December 16, 2024 23:52
@edalzell edalzell marked this pull request as ready for review December 20, 2024 22:51
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