frommeandClaude Opus 5.5
7e182e3d51
The overdue-feedback lists are a live query without stored state, so a feedback nobody will provide any more could not be written off. The new disposition.feedback_waived flag takes a placement out of the admin and manager dashboards, the house-manager list and the reminder mail. Statistics still count a waived placement as a missing feedback. The flag has no UI yet and is set by SQL. Co-Authored-By: Claude Opus 5.5 <[email protected]>
37 lines
1.0 KiB
PHP
37 lines
1.0 KiB
PHP
<?php
|
|
|
|
declare(strict_types=1);
|
|
|
|
namespace DoctrineMigrations;
|
|
|
|
use Doctrine\DBAL\Schema\Schema;
|
|
use Doctrine\Migrations\AbstractMigration;
|
|
|
|
/**
|
|
* Adds the per-placement feedback waiver.
|
|
*
|
|
* The overdue-feedback lists are a live query without any stored state, so there was no way to
|
|
* write off a feedback nobody will provide any more. The flag takes a placement out of the
|
|
* dashboards' overdue lists, the house-manager list and the reminder mail.
|
|
*
|
|
* Written by hand rather than taken from doctrine:migrations:diff, which also carries the
|
|
* pre-existing drift of the dev database.
|
|
*/
|
|
final class Version20261013120000 extends AbstractMigration
|
|
{
|
|
public function getDescription(): string
|
|
{
|
|
return 'Add the feedback_waived flag to dispositions';
|
|
}
|
|
|
|
public function up(Schema $schema): void
|
|
{
|
|
$this->addSql('ALTER TABLE disposition ADD feedback_waived TINYINT DEFAULT 0 NOT NULL');
|
|
}
|
|
|
|
public function down(Schema $schema): void
|
|
{
|
|
$this->addSql('ALTER TABLE disposition DROP feedback_waived');
|
|
}
|
|
}
|