Skip to content

Commit

Permalink
fix: remove query params in create API
Browse files Browse the repository at this point in the history
  • Loading branch information
GloireMutaliko21 committed Apr 29, 2024
1 parent 8d6699e commit 95ff398
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 21 deletions.
7 changes: 2 additions & 5 deletions packages/core/src/tasks/daily-plan/daily-plan.controller.ts
Original file line number Diff line number Diff line change
Expand Up @@ -72,11 +72,8 @@ export class DailyPlanController extends CrudController<DailyPlan> {
@HttpCode(HttpStatus.CREATED)
@Post()
@UseValidationPipe({ transform: true, whitelist: true })
async create(
@Body() entity: CreateDailyPlanDTO,
@Query() options: PaginationParams<DailyPlan>
): Promise<IDailyPlan> {
return await this.dailyPlanService.createDailyPlan(entity, options, entity.taskId);
async create(@Body() entity: CreateDailyPlanDTO): Promise<IDailyPlan> {
return await this.dailyPlanService.createDailyPlan(entity, entity.taskId);
}

/**
Expand Down
17 changes: 1 addition & 16 deletions packages/core/src/tasks/daily-plan/daily-plan.service.ts
Original file line number Diff line number Diff line change
Expand Up @@ -30,11 +30,7 @@ export class DailyPlanService extends TenantAwareCrudService<DailyPlan> {
* @param options
*/

async createDailyPlan(
partialEntity: DeepPartial<IDailyPlan>,
options: PaginationParams<DailyPlan>,
taskId?: ITask['id']
): Promise<IDailyPlan> {
async createDailyPlan(partialEntity: DeepPartial<IDailyPlan>, taskId?: ITask['id']): Promise<IDailyPlan> {
try {
const { employeeId } = partialEntity;
const currentTenantId = RequestContext.currentTenantId();
Expand All @@ -54,17 +50,6 @@ export class DailyPlanService extends TenantAwareCrudService<DailyPlan> {

const query = this.typeOrmRepository.createQueryBuilder(this.tableName);

query.setFindOptions({
...(isNotEmpty(options) &&
isNotEmpty(options.where) && {
where: options.where
}),
...(isNotEmpty(options) &&
isNotEmpty(options.relations) && {
relations: options.relations
})
});

query.where(
new Brackets((qb: WhereExpressionBuilder) => {
qb.andWhere(p(`"${query.alias}"."date" ${likeOperator} :incomingDate`), {
Expand Down

0 comments on commit 95ff398

Please sign in to comment.