feat(squads): squad_applications + ApplyToSquad gated no APTO (#360) - #533
Open
sirelves wants to merge 1 commit into
Open
feat(squads): squad_applications + ApplyToSquad gated no APTO (#360)#533sirelves wants to merge 1 commit into
sirelves wants to merge 1 commit into
Conversation
Abre o único fluxo que o módulo conduz: a candidatura. Entrar exige o gate APTO do onboarding, então quem não concluiu o onboarding Squads é barrado e direcionado para lá. O limite de uma candidatura pendente por squad é o índice parcial UNIQUE (squad_id, user_id) WHERE status = 'pending', não um SELECT prévio: duas candidaturas simultâneas leriam "nenhuma pendente" e inseririam as duas. O insert decide. Uma candidatura recusada sai do índice, então candidatar-se de novo depois continua valendo. A decisão do capitão (aprovar/recusar) fica na #361, e a exclusividade de um squad ativo por pessoa na #362.
5 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Materializa a candidatura a um squad, o único fluxo que o módulo conduz, gated no APTO do onboarding.
squad_applications:status(defaultpending),message,decided_by,decided_at,INDEX (squad_id, status)e o partialUNIQUE (squad_id, user_id) WHERE status = 'pending'SquadApplication(PHPDoc@property,#[Table],#[UseFactory], casts) +SquadApplicationFactory+ enumApplicationStatus(pending/approved/rejected)ApplyToSquad: lêOnboardingCompletionGate::isCompleted($user, OnboardingType::Squads)e barra quem não é APTO comNotAptForSquads, cuja mensagem já nomeia o onboarding exigido para a UI direcionarApplicationAlreadyPendingSobre a duplicata pendente
Não tem
SELECTantes do insert: duas candidaturas simultâneas leriam "nenhuma pendente" e inseririam as duas. Quem decide é o índice parcial, e a Action traduz a violação para a exception de domínio. Como candidatura recusada sai do índice, candidatar-se de novo depois continua valendo (tem teste pra isso).Fora de escopo (por dependência)
DecideApplication) é a feat(squads): DecideApplication (capitão) → membership + evento join #361, do @reag-dev, que sobe em cima desta branchApplyToSquadPlano de testes
ApplyToSquadTest.php: APTO abrependingcom mensagem, não-APTO barrado (sem onboarding, onboarding em progresso, só o Welcome concluído), duplicata pendente barrada, candidatura recusada não bloqueia nova, pendente em outro squad não bloqueia. 7/7 verdesapp-modules/squads: 47/47 verdesmake check(rector + pint + phpstan) limpoCloses #360