Paper 'upload' endpoint implemented #19

Open
danny-mhlv wants to merge 1 commits from feature/upload into develop
danny-mhlv commented 2023-02-26 20:29:00 +00:00 (Migrated from 85.143.176.51:3000)
No description provided.
Klutrem requested changes 2024-06-04 14:24:49 +00:00
Klutrem left a comment
First-time contributor

Пофикси пару вещей и норм

Пофикси пару вещей и норм
@ -83,0 +87,4 @@
/**
* Request handler for PUT /papers/upload
* @param uuid
First-time contributor

а где дексрипшн

а где дексрипшн
@ -0,0 +7,4 @@
* Upload service provider
*/
@Injectable()
export class UploadService {
First-time contributor

зачем отдельный сервис выделять под аплоуд

зачем отдельный сервис выделять под аплоуд
@ -0,0 +18,4 @@
/**
* Elastichsearch server port-number
*/
private readonly ES_PORT = process.env.ES_PORT;
First-time contributor

не надо так напрямую к енвам обращаться из сервиса

не надо так напрямую к енвам обращаться из сервиса
@ -0,0 +31,4 @@
*/
async upload(paperData: PaperDto): Promise<Object | any> {
return new Promise((resolve, reject) => {
try {
First-time contributor

трай кэтч на промисах не работает

трай кэтч на промисах не работает
This pull request can be merged automatically.
You are not authorized to merge this pull request.

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin feature/upload:feature/upload
git checkout feature/upload
Sign in to join this conversation.
No Reviewers
No Label
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Scipaper/Backend#19
No description provided.