Appwrite/appwrite
1
0
Fork 0
mirror of https://github.com/appwrite/appwrite.git synced 2026-05-26 13:51:13 +00:00
Code Issues Packages Projects Releases Wiki Activity

refactor(insights): nest CTA execution and dismissal as sub-resources

Browse Source 
Move POST /v1/insights/:id/dismiss to /v1/insights/:id/dismissals
and POST /v1/insights/:id/ctas/:ctaId/trigger to
/v1/insights/:id/ctas/:ctaId/executions, with the corresponding
class moves into Http/Insights/Dismissal/Create.php and
Http/CTA/Execution/Create.php. Rename the response model to
InsightCtaExecution and update events.php to surface dismissal
and execution as resource events with create verbs. The reshape
matches the rest of the API where verbs hang off plural sub-resources.
This commit is contained in:
Jake Barnby
2026-05-01 14:04:31 +12:00
parent d15be21923
commit da5a137b98
7 changed files with 45 additions and 36 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/Appwrite/Platform/Modules/Insights/Http/Cta/Trigger.php → src/Appwrite/Platform/Modules/Insights/Http/CTA/Execution/Create.php
+11 -11
View File
@@ -1,6 +1,6 @@
<?php
namespace Appwrite\Platform\Modules\Insights\Http\Cta;
namespace Appwrite\Platform\Modules\Insights\Http\CTA\Execution;
use Appwrite\Event\Database as EventDatabase;
use Appwrite\Event\Event;
@@ -19,26 +19,26 @@ use Utopia\Platform\Scope\HTTP;
use Utopia\Registry\Registry as UtopiaRegistry;
use Utopia\Validator\Text;
class Trigger extends Action
class Create extends Action
{
use HTTP;
public static function getName()
{
return 'triggerInsightCta';
return 'createInsightCtaExecution';
}
public function __construct()
{
$this
->setHttpMethod(Action::HTTP_REQUEST_METHOD_POST)
->setHttpPath('/v1/insights/:insightId/ctas/:ctaId/trigger')
->desc('Trigger insight CTA')
->setHttpPath('/v1/insights/:insightId/ctas/:ctaId/executions')
->desc('Create insight CTA execution')
->groups(['api', 'insights'])
->label('scope', 'insights.write')
->label('event', 'insights.[insightId].ctas.[ctaId].trigger')
->label('event', 'insights.[insightId].ctas.[ctaId].executions.create')
->label('resourceType', RESOURCE_TYPE_INSIGHTS)
->label('audits.event', 'insight.cta.trigger')
->label('audits.event', 'insight.cta.execution.create')
->label('audits.resource', 'insight/{request.insightId}')
->label('abuse-key', 'projectId:{projectId},userId:{userId}')
->label('abuse-limit', APP_LIMIT_WRITE_RATE_DEFAULT)
@@ -46,15 +46,15 @@ class Trigger extends Action
->label('sdk', new Method(
namespace: 'insights',
group: 'insights',
name: 'triggerCta',
name: 'createCtaExecution',
description: <<<EOT
Trigger a CTA attached to an insight. Looks up the registered server-side action by name, validates the CTA params, and executes the action on behalf of the caller.
Execute a CTA attached to an insight. Looks up the registered server-side action by name, validates the CTA params, and runs the action on behalf of the caller.
EOT,
auth: [AuthType::ADMIN, AuthType::KEY],
responses: [
new SDKResponse(
code: Response::STATUS_CODE_OK,
model: Response::MODEL_INSIGHT_CTA_RESULT,
model: Response::MODEL_INSIGHT_CTA_EXECUTION,
),
]
))
@@ -167,6 +167,6 @@ class Trigger extends Action
'action' => $actionName,
'status' => $status,
'result' => $resultPayload,
]), Response::MODEL_INSIGHT_CTA_RESULT);
]), Response::MODEL_INSIGHT_CTA_EXECUTION);
}
}
src/Appwrite/Platform/Modules/Insights/Http/Insights/Dismiss.php → src/Appwrite/Platform/Modules/Insights/Http/Insights/Dismissal/Create.php
+8 -8
View File
@@ -1,6 +1,6 @@
<?php
namespace Appwrite\Platform\Modules\Insights\Http\Insights;
namespace Appwrite\Platform\Modules\Insights\Http\Insights\Dismissal;
use Appwrite\Event\Event;
use Appwrite\Extend\Exception;
@@ -15,26 +15,26 @@ use Utopia\Database\Validator\UID;
use Utopia\Platform\Action;
use Utopia\Platform\Scope\HTTP;
class Dismiss extends Action
class Create extends Action
{
use HTTP;
public static function getName()
{
return 'dismissInsight';
return 'createInsightDismissal';
}
public function __construct()
{
$this
->setHttpMethod(Action::HTTP_REQUEST_METHOD_POST)
->setHttpPath('/v1/insights/:insightId/dismiss')
->desc('Dismiss insight')
->setHttpPath('/v1/insights/:insightId/dismissals')
->desc('Create insight dismissal')
->groups(['api', 'insights'])
->label('scope', 'insights.write')
->label('event', 'insights.[insightId].dismiss')
->label('event', 'insights.[insightId].dismissals.create')
->label('resourceType', RESOURCE_TYPE_INSIGHTS)
->label('audits.event', 'insight.dismiss')
->label('audits.event', 'insight.dismissal.create')
->label('audits.resource', 'insight/{response.$id}')
->label('abuse-key', 'projectId:{projectId},userId:{userId}')
->label('abuse-limit', APP_LIMIT_WRITE_RATE_DEFAULT)
@@ -42,7 +42,7 @@ class Dismiss extends Action
->label('sdk', new Method(
namespace: 'insights',
group: 'insights',
name: 'dismiss',
name: 'createDismissal',
description: <<<EOT
Dismiss an insight. Stamps the current user and time on the insight without deleting it, so analyzers can see it has been acknowledged.
EOT,
src/Appwrite/Platform/Modules/Insights/Services/Http.php
+4 -5
View File
@@ -2,10 +2,10 @@
namespace Appwrite\Platform\Modules\Insights\Services;
use Appwrite\Platform\Modules\Insights\Http\Cta\Trigger as TriggerCta;
use Appwrite\Platform\Modules\Insights\Http\CTA\Execution\Create as CreateInsightCtaExecution;
use Appwrite\Platform\Modules\Insights\Http\Insights\Create as CreateInsight;
use Appwrite\Platform\Modules\Insights\Http\Insights\Delete as DeleteInsight;
use Appwrite\Platform\Modules\Insights\Http\Insights\Dismiss as DismissInsight;
use Appwrite\Platform\Modules\Insights\Http\Insights\Dismissal\Create as CreateInsightDismissal;
use Appwrite\Platform\Modules\Insights\Http\Insights\Get as GetInsight;
use Appwrite\Platform\Modules\Insights\Http\Insights\Update as UpdateInsight;
use Appwrite\Platform\Modules\Insights\Http\Insights\XList as ListInsights;
@@ -21,9 +21,8 @@ class Http extends Service
$this->addAction(GetInsight::getName(), new GetInsight());
$this->addAction(ListInsights::getName(), new ListInsights());
$this->addAction(UpdateInsight::getName(), new UpdateInsight());
$this->addAction(DismissInsight::getName(), new DismissInsight());
$this->addAction(DeleteInsight::getName(), new DeleteInsight());
$this->addAction(TriggerCta::getName(), new TriggerCta());
$this->addAction(CreateInsightDismissal::getName(), new CreateInsightDismissal());
$this->addAction(CreateInsightCtaExecution::getName(), new CreateInsightCtaExecution());
}
}
src/Appwrite/Utopia/Response.php
+1 -1
View File
@@ -334,7 +334,7 @@ class Response extends SwooleResponse
public const MODEL_INSIGHT = 'insight';
public const MODEL_INSIGHT_LIST = 'insightList';
public const MODEL_INSIGHT_CTA = 'insightCta';
public const MODEL_INSIGHT_CTA_RESULT = 'insightCtaResult';
public const MODEL_INSIGHT_CTA_EXECUTION = 'insightCtaExecution';
// Console
public const MODEL_CONSOLE_VARIABLES = 'consoleVariables';
src/Appwrite/Utopia/Response/Model/InsightCtaResult.php → src/Appwrite/Utopia/Response/Model/InsightCtaExecution.php
+5 -5
View File
@@ -5,20 +5,20 @@ namespace Appwrite\Utopia\Response\Model;
use Appwrite\Utopia\Response;
use Appwrite\Utopia\Response\Model;
class InsightCtaResult extends Model
class InsightCtaExecution extends Model
{
public function __construct()
{
$this
->addRule('insightId', [
'type' => self::TYPE_STRING,
'description' => 'ID of the insight the CTA was triggered against.',
'description' => 'ID of the insight the CTA was executed against.',
'default' => '',
'example' => '5e5ea5c16897e',
])
->addRule('ctaId', [
'type' => self::TYPE_STRING,
'description' => 'ID of the CTA that was triggered.',
'description' => 'ID of the CTA that was executed.',
'default' => '',
'example' => 'createIndex',
])
@@ -44,11 +44,11 @@ class InsightCtaResult extends Model
public function getName(): string
{
return 'InsightCtaResult';
return 'InsightCtaExecution';
}
public function getType(): string
{
return Response::MODEL_INSIGHT_CTA_RESULT;
return Response::MODEL_INSIGHT_CTA_EXECUTION;
}
}