apache/openwhisk

List Rule needs status field.

Open

#4.438 geöffnet am 11. Apr. 2019

Auf GitHub ansehen
 (5 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Scala (6.777 Stars) (1.177 Forks)batch import
APIenhancementhelp wanted

Beschreibung

Hi Guys, As I know, the response data when get list rule like below so we need return response for list rule with status(active/inactive) for each object rule.

[
        {
            "action": {
                "name": "action01",
                "path": "myspace"
            },
            "annotations": [],
            "name": "action01_trigger1",
            "namespace": "myspace",
            "publish": false,
            "trigger": {
                "name": "trigger1",
                "path": "myspace"
            },
            "version": "0.0.1"
          **//missing status ???**
        }

    ]

Any help for improve this? Thanks

Contributor Guide

Tech Stack
scala
Domain
backendapi
Issue Type
feature
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
under 1 hour
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
Basic Scala knowledgeFamiliarity with REST API conceptsUnderstanding of OpenWhisk rule model
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60
Research-Richtung
Look at the rule listing endpoint in the OpenWhisk controller, likely in the RulesApi or RulesResource Scala file. The response currently lacks a 'status' field; you need to add the rule's status (active/inactive) to the JSON serialization of the Rule object. Check the Rule case class and its JSON serializer to include the status field.