-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
10 naming #25
10 naming #25
Changes from all commits
3d872c5
a9d954f
ce9cc38
481c5da
b0de70a
c4176ce
5c90e51
a2d989b
edbd119
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,6 +1,6 @@ | ||
export * from './auth.decorators'; | ||
export * from './event.decorators'; | ||
export * from './notes.decorators'; | ||
export * from './note.decorators'; | ||
export * from './search.decorators'; | ||
export * from './task.decorators'; | ||
export * from './user.decorators'; |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -20,7 +20,7 @@ import { JwtGuard } from 'src/auth/guard'; | |
import { GetUser } from 'src/decorators'; | ||
import { | ||
AddEventExceptionApi, | ||
CreateEventByCurrentUserApi, | ||
AddEventApi, | ||
DeleteEventApi, | ||
DeleteExceptionApi, | ||
EditEventApi, | ||
|
@@ -54,7 +54,7 @@ export class EventController { | |
async getUserEvents( | ||
@GetUser('id') userId: number, | ||
): Promise<ReturnEventDTO[]> { | ||
return await this.eventService.fetchUserEvents(userId); | ||
return await this.eventService.fetchByUser(userId); | ||
} | ||
|
||
@Get('dates') | ||
|
@@ -86,12 +86,12 @@ export class EventController { | |
} | ||
|
||
@Post() | ||
@CreateEventByCurrentUserApi() | ||
async createEventByCurrentUser( | ||
@AddEventApi() | ||
async addEvent( | ||
@GetUser('id') userId: number, | ||
@Body() eventDto: CreateEventDTO, | ||
): Promise<ReturnEventDTO> { | ||
return await this.eventService.createEvent(userId, eventDto); | ||
return await this.eventService.add(userId, eventDto); | ||
} | ||
|
||
@Post('exceptions/:id') | ||
|
@@ -100,7 +100,7 @@ export class EventController { | |
@Param('id') eventId: number, | ||
@Body() exceptionDto: CreateEventExceptionDTO, | ||
): Promise<ReturnEventExceptionDTO> { | ||
return await this.eventService.createException(eventId, exceptionDto); | ||
return await this.eventService.addException(eventId, exceptionDto); | ||
} | ||
|
||
@Patch(':id') | ||
|
@@ -109,7 +109,7 @@ export class EventController { | |
@Param('id') eventId: number, | ||
@Body() editEventDto: EditEventDTO, | ||
): Promise<ReturnEventDTO> { | ||
return await this.eventService.editById(eventId, editEventDto); | ||
return await this.eventService.edit(eventId, editEventDto); | ||
} | ||
|
||
@Patch('exceptions/:id') | ||
|
@@ -118,7 +118,7 @@ export class EventController { | |
@Param('id') exceptionId: number, | ||
@Body() editExceptionDto: EditEventExceptionDTO, | ||
): Promise<ReturnEventExceptionDTO> { | ||
return await this.eventService.editExceptionById( | ||
return await this.eventService.editException( | ||
exceptionId, | ||
editExceptionDto, | ||
); | ||
|
@@ -128,13 +128,13 @@ export class EventController { | |
@Delete(':id') | ||
@DeleteEventApi() | ||
async deleteEvent(@Param('id') eventId: number): Promise<void> { | ||
await this.eventService.deleteById(eventId); | ||
await this.eventService.delete(eventId); | ||
} | ||
|
||
@HttpCode(HttpStatus.NO_CONTENT) | ||
@Delete('exceptions/:id') | ||
@DeleteExceptionApi() | ||
async deleteException(@Param('id') exceptionId: number): Promise<void> { | ||
await this.eventService.removeExceptionById(exceptionId); | ||
await this.eventService.deleteException(exceptionId); | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. deleteException in this case is clear, because we know that we want to delete one so it is obvious that we need for example id |
||
} | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,2 +1,2 @@ | ||
export * from './notes.dto'; | ||
export * from './note.dto'; | ||
export * from './editNote.dto'; |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -7,16 +7,16 @@ import { | |
Post, | ||
UseGuards, | ||
} from '@nestjs/common'; | ||
import { NoteService } from './notes.service'; | ||
import { NoteService } from './note.service'; | ||
import { editNoteDTO, NoteDTO } from './dto'; | ||
import { JwtGuard } from '../../auth/guard'; | ||
import { GetUser } from '../../decorators'; | ||
import { UserEntity } from 'src/database/entities/user.entity'; | ||
import { ApiBearerAuth, ApiTags } from '@nestjs/swagger'; | ||
import { | ||
AddNoteApi, | ||
EditNoteByIdApi, | ||
FetchUserNotesApi, | ||
EditNoteApi, | ||
GetUserNotesApi | ||
} from '../../decorators/OpenAPI'; | ||
|
||
@UseGuards(JwtGuard) | ||
|
@@ -27,20 +27,20 @@ export class NoteController { | |
constructor(private notesService: NoteService) {} | ||
|
||
@Get() | ||
@FetchUserNotesApi() | ||
fetchUserNotes(@GetUser() user: UserEntity) { | ||
return this.notesService.fetchUserNotes(user); | ||
@GetUserNotesApi() | ||
getUserNotes(@GetUser() user: UserEntity): Promise<NoteDTO[]> { | ||
return this.notesService.fetchByUser(user); | ||
Comment on lines
+30
to
+32
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I forgot to mention that if TypeScript infer type properly if you hover your mouse on function then you don't need to declare type explicitly |
||
} | ||
|
||
@Post(':id') | ||
@Post() | ||
@AddNoteApi() | ||
addNote(@GetUser() user: UserEntity, @Body() noteDto: NoteDTO) { | ||
return this.notesService.addNote(user, noteDto); | ||
addNote(@GetUser() user: UserEntity, @Body() noteDto: NoteDTO): Promise<NoteDTO> { | ||
return this.notesService.add(user, noteDto); | ||
} | ||
|
||
@Patch(':id') | ||
@EditNoteByIdApi() | ||
editNoteById(@Param('id') id: number, @Body() noteDto: editNoteDTO) { | ||
return this.notesService.editNoteById(id, noteDto); | ||
@EditNoteApi() | ||
editNote(@Param('id') id: number, @Body() noteDto: editNoteDTO): Promise<void> { | ||
return this.notesService.edit(id, noteDto); | ||
} | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And that one is clear, cool