ehrbase/documentation

The documentation should state that the AQL endpoint includes some URL parameters otherwise using the client example wont work.

SevKohler opened this issue · 0 comments

Just tried out the AQL endpoint.
The documentation states here how to setup the client: https://ehrbase.readthedocs.io/en/latest/04_sdk/02_reference/01_client_module/01_client/index.html
i copy pasta and then used the AQL endpoint, which duplicates (/rest/openehr/v1/) therefore the AQLs wont work since they will be executed on /ehrbase/rest/openehr/rest/openehr/v1/query/aql/ .
This quickly leads to confusion especially for new users.
Therefore shouldnt the client point to "http://localhost:8080/ehrbase/" ?

Also adding the credentials and so on are not explained (afaik) so adding smt like that would be good or at least explain it (maybe you guys did and i just missed it).

        CredentialsProvider credentialsProvider = new BasicCredentialsProvider();
        credentialsProvider.setCredentials(AuthScope.ANY, new UsernamePasswordCredentials("*****", "**********"));
        CloseableHttpClient httpClient = HttpClientBuilder.create()
                .setDefaultCredentialsProvider(credentialsProvider)
                .build();
                
        DefaultRestClient client = ...

PS: Add maybe a small hint that the template provider can be null if using only the AQL endpoint (does no harm before users start configuring a TP they dont need).