nurlandadasev
commented
3 years ago Добры день! Хорошо я сделаю изменения над своим кодом.
28 мая 2021 г., в 14:55, Meiendorf @.***> написал(а):
Добрый день. Меня зовут Роман, мы, совместно с @aandrey1993 https://github.com/aandrey1993 проводим ревью тестового задания. По ходу ревью будем открывать тикеты с вопросами/предложениями для улучшения кода.
Описание
В запросе на обновление баланса к балансу прибавляется значение, полученное от клиента, вместо перезаписывания баланса полностью: https://github.com/nurlandadasev/mobile-banking-app/blob/d2b87c776864f568ae5d211b8d456772533e3aec/src/main/java/com/ma/mobilebankingapp/services/impl/AccountServiceImpl.java#L91 https://github.com/nurlandadasev/mobile-banking-app/blob/d2b87c776864f568ae5d211b8d456772533e3aec/src/main/java/com/ma/mobilebankingapp/services/impl/AccountServiceImpl.java#L91 Это немного не интуитивное поведение. Нужно либо в описании к запросу об этом написать, либо полностью перезаписывать значение баланса.
— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/nurlandadasev/mobile-banking-app/issues/1, or unsubscribe https://github.com/notifications/unsubscribe-auth/ASDU4EOCYZN33KFZTOLOHTLTP6AEVANCNFSM45WONTFQ.
Добрый день. Меня зовут Роман, мы, совместно с @aandrey1993 проводим ревью тестового задания. По ходу ревью будем открывать тикеты с вопросами/предложениями для улучшения кода.
Описание
В запросе на обновление баланса к балансу прибавляется значение, полученное от клиента, вместо перезаписывания баланса полностью: https://github.com/nurlandadasev/mobile-banking-app/blob/d2b87c776864f568ae5d211b8d456772533e3aec/src/main/java/com/ma/mobilebankingapp/services/impl/AccountServiceImpl.java#L91
Это немного не интуитивное поведение. Нужно либо в описании к запросу об этом написать, либо полностью перезаписывать значение баланса.