Since the work on these is almost done, only one issue will be allocated for them

• About
• App
• Content library
• Custom property
• Engine
• Extension
• Stream
• System rule
• Table
• Tag
• Task

Provide the option to append tags values. At the moment all existing values will be overritten with the provided values

At the moment all update methods are returning only the update response status (usually 200). For conistency all these methids should return the updated object data as well

At the moment all xxxGet requests accept id as optional parameter. If id is NOT specified the method returns IxxxCondensed[] if not Ixxx[].
Dont like methods to return different data formats.

For this reason xxxGet should require id parameter as mandatory and return Ixxx (no array)
And new methods should be implemented xxxGetAll (without any params) which will return IxxxCondensed[]

All code parts marked as TODO and REVIEW to be resolved and cleared

When id is provided Qlik returns the full object
If id is not provided then Qlik returns array of condensed objects

In relation with #43 and #44 all object should provide additional parameter(s)

The following files to be updated:

Once the node is created how to call the certificateSetup?

Once v1 of the package is released, all test are written and all (known) bugs are resolved ... can think of:

• expose additional methods. At v1 the package will contain only the "main" methods. The Repo API contains more methods that can be exposed.
• extra methods can be added. These methods do not exists in the Repo API directly and will be based on combination of the "main" methods

All classes should have xxxRemoveFilter method implemented

At the moment all the methods are called via repoApi.xxx. For example: repoApi.appGet() or repoApi.streamGet() etc.

Think it will be better to change this to: repoApi.App.Get(), repoApi.Stream.Get() etc

Is it possible at all?

For consistencty the arguments for all methods should be an objects. At the moment some methods are accepting named arguments

Investigate if all required methods have update method

Since the codebase is changed all thrown error messages should be changed to reflect it

Method name
reloadTask.create and reloadTask.upadate

Describe the issue
At the moment both methods accept only appId. If appFilter is provided the methods will search for app, based on the filter, and get its ID from the response. If the app filter returns more than one app or 0 app the methods will throw an error. If appId and appFilter are provided then appId is with priority

Code example
n/a

Possible alternatives?
n/a

Additional context
n/a

Some documentation mismatch. Need more investigation about the parameters

Pem; should be "Pem"

At the moment each object method will return an extra data - baseUrl and repoClient values. These are not required and should be excluded. The return data should be only relevant to the object itself. baseUrl is informative but essentuially useles. repoClient also is irrelevant in the context of the object. repoClient instance is the one running the code and there is no need to be returned in each object instance

• IHttpReturnRemove - the name should be different. It doesent make sense anymore
• IRemoveFilter - this can be removed. Essentially it can be replaced by IHttpReturnRemove[]

Name should "use letters and numbers".

Check if the name is only letters and numbers. If not - throw an error

All xxxGet have to return an array. If id is provided then force to return array with one element

• tests for all phases are completed
• no outstanding or known bugs/TODO/REVIEW

Why after building the genericClient is undefined?

The interfaces can be organized bit better. There are a lot of repetition in them as well

Provide the option to append customer properties values. At the moment all existing values will be overritten with the provided values

Method name
reloadTask.addTriggerMany and reloadTask.addTriggerSchema

Describe the issue
When specifying Weekly repeat, the function that is "converting" the day names to day numbers is incorrectly referenced

Code example
n/a

Possible alternatives?
n/a

Additional context
n/a

virtualProxies.create is not passing name property when making the request. name is not mandatory parameter in QRS API but it should be here.

At the moment each object class expose its methods. For example:

• Tag
  • create
  • get
  • remove
  • update
  • getFilter
  • ...

The new code change will (sometimes) return new instance of class and expose methods relevant to the object. For example:

• Tags
  • remove - returns the data
  • removeFilter - returns data
  • get - returns new instance of the Tag class
  • create - returns new instance of the Tag class
  • getFilter - returns array of instances of the Tag class

The instance of the Tag class will have its own methods:

• remove
• update

So ... the old way code was:

const tag = await repoApi.tag.get("id-goes-here");
// tag variable will have the details of the tag: id, name, createdDate, etc
const createTag = await repoApi.tag.create("id-goes-here");
// createTag variable will have the details of the tag: id, name, createdDate, etc
const removeTag = await repoApi.tag.create("id-goes-here");
// removeTag variable will have the response details

the new way:

const tag = await repoApi.tag.get("id-goes-here");
// at this point the tag variable will expose the details of the tag and some methods
// tag.details - id, name, createdDate etc
// await tag.remove() - method that will remove the tag. no need to specify the id
// await tag.update("new tag name") - method to update the name of the tag. id is not needed

methods that are returning array of instances:

const allTags = await repoApi.tag.getAll();
// each element of the array exposes Tag class instance
// allTags[0].details
// await allTags[0].remove()
// await allTags[0].update("new tag name")

an example how to remove multiple tags:

// get all tags which name starts with "test"
const filterTags = await repoApi.tag.getFilter(`name sw 'test'`);
const removeResp = await Promise.all(
  return filterTags.map(tag => tag.remove())
);
// removeResp will be an array of remove responses:
// [ 
//    {id: "some-tag-id", status: 204},
//    {id: "some-other-tag-id", status: 204},
//    ...
// ]

The change will not affect some objects. Since not all objects are required to return class instance. Example objects/endppoints About and ServiceStatus

Method name
extension.import

Describe the issue
The method should accept tags and custom properties values. At the moment it supports only the required params (file and password)

Code example
n/a

Possible alternatives?
n/a

Additional context
n/a