mirror of
https://github.com/appwrite/appwrite
synced 2026-05-24 09:28:40 +00:00
Merge pull request #10547 from appwrite/ser-409
fix: Send deployment ready event after updating associated resource
This commit is contained in:
commit
3459133fd4
1 changed files with 5 additions and 4 deletions
|
|
@ -1107,10 +1107,6 @@ class Builds extends Action
|
||||||
$resource = $dbForProject->updateDocument($resource->getCollection(), $resource->getId(), new Document(['latestDeploymentStatus' => $deployment->getAttribute('status', '')]));
|
$resource = $dbForProject->updateDocument($resource->getCollection(), $resource->getId(), new Document(['latestDeploymentStatus' => $deployment->getAttribute('status', '')]));
|
||||||
}
|
}
|
||||||
|
|
||||||
$queueForRealtime
|
|
||||||
->setPayload($deployment->getArrayCopy())
|
|
||||||
->trigger();
|
|
||||||
|
|
||||||
if ($isVcsEnabled) {
|
if ($isVcsEnabled) {
|
||||||
$this->runGitAction('ready', $github, $providerCommitHash, $owner, $repositoryName, $project, $resource, $deployment->getId(), $dbForProject, $dbForPlatform, $queueForRealtime);
|
$this->runGitAction('ready', $github, $providerCommitHash, $owner, $repositoryName, $project, $resource, $deployment->getId(), $dbForProject, $dbForPlatform, $queueForRealtime);
|
||||||
}
|
}
|
||||||
|
|
@ -1198,6 +1194,11 @@ class Builds extends Action
|
||||||
Console::log('Deployment activated');
|
Console::log('Deployment activated');
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// Send realtime event after updating the associated resource so that Console will have the resource's deployment details when re-fetching.
|
||||||
|
$queueForRealtime
|
||||||
|
->setPayload($deployment->getArrayCopy())
|
||||||
|
->trigger();
|
||||||
|
|
||||||
if ($resource->getCollection() === 'sites') {
|
if ($resource->getCollection() === 'sites') {
|
||||||
// VCS branch
|
// VCS branch
|
||||||
$branchName = $deployment->getAttribute('providerBranch');
|
$branchName = $deployment->getAttribute('providerBranch');
|
||||||
|
|
|
||||||
Loading…
Reference in a new issue