Protei/PromucFlow_constructor
2
0
Fork 0
Code Issues Pull Requests Actions 2 Packages Projects Releases Wiki Activity

Merge branch 'bug/missing-response-meta-in-test-api' into 'release'

Browse Source 
Fix missing `responseMeta` in response for /datasources/test endpoint.

See merge request theappsmith/internal-tools-server!291
This commit is contained in:
Shrikant Kandula 2020-04-20 12:40:42 +00:00
commit 02a384698b
1 changed files with 2 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
app/server/appsmith-server/src/main/java/com/appsmith/server/controllers/DatasourceController.java
View File

@ -6,6 +6,7 @@ import com.appsmith.server.domains.Datasource;
import com.appsmith.server.dtos.ResponseDTO;
import com.appsmith.server.services.DatasourceService;
import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.http.HttpStatus;
import org.springframework.util.CollectionUtils;
import org.springframework.web.bind.annotation.PostMapping;
import org.springframework.web.bind.annotation.RequestBody;
@ -41,10 +42,6 @@ public class DatasourceController extends BaseController<DatasourceService, Data
return Mono.just(new DatasourceTestResult(datasource1.getInvalids()));
}
})
.map(testResult -> {
ResponseDTO<DatasourceTestResult> response = new ResponseDTO<>();
response.setData(testResult);
return response;
});
.map(testResult -> new ResponseDTO<>(HttpStatus.OK.value(), testResult, null));
}
}