Syspeople / CodeNow

Empower your ServiceNow Development with no instance configuration.
MIT License
8 stars 7 forks source link

Add angular provider #96

Closed CGraabaek closed 5 years ago

CGraabaek commented 5 years ago

Jeg har løst det nu .

Har et par ting jeg godt vil have din mening om:

                            switch (recordtype)
                            {
                                case "Angular Provider": {
                                    vscode.window.showQuickPick(["Directive", "Service", "Factory"], {
                                        placeHolder: "Choose Type"
                                    }).then((item) =>
                                    {
                                        let r = instance.CreateRecord(SupportedRecords[recordtype], {
                                            'name': name,
                                            'type': item
                                        });

                                        //@ts-ignore already null checked
                                        r.then((newRecord) =>
                                        {
                                            wm.AddRecord(newRecord, instance);
                                        }).catch((err) =>
                                        {
                                            vscode.window.showErrorMessage(err);
                                        });
                                    });
                                    break;
                                }
                                default: {
                                    let r = instance.CreateRecord(SupportedRecords[recordtype], {
                                        'name': name
                                    });
                                    //@ts-ignore already null checked

                                    r.then((newRecord) =>
                                    {
                                        wm.AddRecord(newRecord, instance);
                                    }).catch((err) =>
                                    {
                                        vscode.window.showErrorMessage(err);
                                    });
                                    break;
                                }
                            }

Og kan ikke helt få lavet de der Promises korrekt.

Tænker den stadig godt kan merges også kan det optimeres i et andet PR.

CGraabaek commented 5 years ago

Havde lavet update-set før det blev lavet generisk record getter, har dog fikset det i merge-conflicten.