Skip to content

Commit

Permalink
delete unused routes (#125)
Browse files Browse the repository at this point in the history
  • Loading branch information
thozh authored Jul 23, 2020
1 parent cb61d6d commit 86379ea
Show file tree
Hide file tree
Showing 5 changed files with 9 additions and 33 deletions.
19 changes: 0 additions & 19 deletions api/src/pull-request/pull-request.controller.ts

This file was deleted.

3 changes: 1 addition & 2 deletions api/src/pull-request/pull-request.module.ts
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@ import { Module, forwardRef } from '@nestjs/common';
import { TypeOrmModule } from '@nestjs/typeorm';
import { PullRequest } from './pull-request.entity';
import { PullRequestService } from './pull-request.service';
import { PullRequestController } from './pull-request.controller';
import { RepositoryModule } from '../repository/repository.module';

@Module({
Expand All @@ -12,6 +11,6 @@ import { RepositoryModule } from '../repository/repository.module';
],
providers: [PullRequestService],
exports: [PullRequestService, TypeOrmModule],
controllers: [PullRequestController],
controllers: [],
})
export class PullRequestModule {}
5 changes: 4 additions & 1 deletion api/src/repository/repository.controller.ts
Original file line number Diff line number Diff line change
Expand Up @@ -48,6 +48,9 @@ import { RepositoryService } from './repository.service';
pullRequests: {},
},
},
routes: {
only: ['getManyBase', 'getOneBase', 'deleteOneBase'],
},
})
@CrudAuth({
property: 'user',
Expand Down Expand Up @@ -257,7 +260,7 @@ export class RepositoryController implements CrudController<Repository> {

const repositories = await this.service.getAllRepos();
const userRepos = repositories.filter((repo: Repository) =>
repo.users.some((repoUser) => repoUser.id === user.id),
repo.users.some(repoUser => repoUser.id === user.id),
);
const nbUserRepos = userRepos.length;

Expand Down
9 changes: 1 addition & 8 deletions api/src/users/users.controller.ts
Original file line number Diff line number Diff line change
Expand Up @@ -3,22 +3,15 @@ import {
NotFoundException,
Param,
Post,
UseGuards,
Req,
UnauthorizedException,
UseGuards,
} from '@nestjs/common';
import { AuthGuard } from '@nestjs/passport';
import { Crud } from '@nestjsx/crud';
import { Repository } from '../repository/repository.entity';
import { RepositoryService } from '../repository/repository.service';
import { User } from './user.entity';
import { UsersService } from './users.service';

@Crud({
model: {
type: User,
},
})
@Controller('users')
export class UsersController {
constructor(
Expand Down
6 changes: 3 additions & 3 deletions api/src/webhooks/webhooks.controller.ts
Original file line number Diff line number Diff line change
Expand Up @@ -39,8 +39,8 @@ export class WebhooksController {

if (user) {
const repos = await this.repositoryService.getAllRepos();
const nbRepos = repos.filter((repo) =>
repo.users.some((repoUser) => repoUser.id === user.id),
const nbRepos = repos.filter(repo =>
repo.users.some(repoUser => repoUser.id === user.id),
).length;
let repositories = [];
let repositoriesRemoved = [];
Expand Down Expand Up @@ -78,7 +78,7 @@ export class WebhooksController {
);*/

await Promise.all(
repositoriesRemoved.map((repoAdd) => {
repositoriesRemoved.map(repoAdd => {
return this.repositoryService.deleteRepo(
{
githubId: repoAdd.id,
Expand Down

0 comments on commit 86379ea

Please sign in to comment.