a2n-seattle / a2n-seattle/rms-app

[Tech debt] In BorrowBatch, use BorrowItem.execute to borrow items

Aberta
#253 1 comentário 0 reações 0 responsáveis Ver no GitHub
Tech Debt
Linguagem predominante
TypeScript
Estrelas
1
Forks
1
Merge médio
27min
PRs com merge (30d)
4

Descrição

# Why?

Currently, we are directly calling ItemTable database functions for BorrowBatch. It is missing field validations for each item in the batch that would be borrowed

# What?

After getting a list of items in the batch, call BorrowItem API to borrow the list of items, instead of calling itemTable.changeBorrower

# Additional context

Guia de contribuição

Nenhum guia de contribuição indexado para este repositório

Direção de pesquisa

Encontre o componente ou serviço BorrowBatch e localize onde ele chama as funções de banco de dados de ItemTable. Em seguida, encontre a API BorrowItem (provavelmente um serviço ou função) que lida com o empréstimo de um único item com validações. A tarefa é substituir as chamadas diretas ao banco de dados por chamadas a BorrowItem.execute para cada item do lote. Verifique os testes existentes relacionados a BorrowBatch ou BorrowItem para entender o comportamento esperado.

Escrita pelo modelo de indexação a partir do texto da issue.

Avaliação

Stack de tecnologia
typescript
Domínio
api, backend
Tipo de issue
Refatoração
Dificuldade
3/5
Tempo estimado
1-2 dias
Status de atividade
Estagnada
Clareza
Razoavelmente clara
Facilidade para iniciantes
45/100

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.