Skip to content

Add migration, models, factories, and tests for review submissions - #1260

Merged
outdooracorn merged 3 commits into
mainfrom
ollie-review-submission-migrations-and-models-T433682
Sep 21, 2026
Merged

outdooracorn merged 3 commits into
mainfrom
ollie-review-submission-migrations-and-models-T433682

Conversation

@outdooracorn

Copy link
Copy Markdown
Member

Bug: T433682

@outdooracorn
outdooracorn added this pull request to stack #1262 September 21, 2026 08:33
Comment thread app/ReviewSubmission.php Outdated
Comment thread database/migrations/2026_08_28_081328_create_review_submission_actions_table.php Outdated
Comment thread database/seeds/ReviewSubmissionSeeder.php
Comment thread database/factories/ReviewSubmissionActionFactory.php Outdated
Comment thread app/ReviewSubmission.php Outdated
Comment thread app/Enums/ReviewSubmissionActionType.php Outdated
@outdooracorn
outdooracorn force-pushed the ollie-review-submission-migrations-and-models-T433682 branch 4 times, most recently from b0c8c55 to 0ff71cb Compare September 21, 2026 12:39
@outdooracorn
outdooracorn force-pushed the ollie-review-submission-migrations-and-models-T433682 branch from 0ff71cb to 6f9d3cb Compare September 21, 2026 12:58
Comment thread tests/Feature/Models/ReviewSubmissionTest.php

@tarrow tarrow left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pair reviewed with @outdooracorn . Looks good to me. We really want this going and are happy to follow up a bit more later.

Comment thread tests/Feature/Models/ReviewSubmissionActionTest.php
@outdooracorn
outdooracorn merged commit 3cc2c8e into main Sep 21, 2026
4 checks passed
@outdooracorn
outdooracorn deleted the ollie-review-submission-migrations-and-models-T433682 branch September 21, 2026 13:36
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