MinterTeam / minter-console-web

Official Minter Console website
https://console.minter.network
MIT License
35 stars 22 forks source link

Небольшой патч который позволит автоматически заполнять из query поля address, amount и message в форме отправки монет #8

Closed abrikos closed 5 years ago

abrikos commented 5 years ago
Index: components/CoinSendForm.vue
IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
===================================================================
--- components/CoinSendForm.vue (revision b9166737e8d23b7ef75ceda49ab854cf236f306d)
+++ components/CoinSendForm.vue (date 1560354470000)
@@ -18,6 +18,12 @@
     import InputUppercase from '~/components/common/InputUppercase';
     import ButtonCopyIcon from '~/components/common/ButtonCopyIcon';
     import Modal from '~/components/common/Modal';
+    import url from 'url';
+    import qs from 'qs';
+
+
+    const queryParams = qs.parse(url.parse(document.location.href).query);
+

     export default {
         components: {
@@ -43,8 +49,8 @@
                 serverSuccess: '',
                 form: {
                     nonce: '',
-                    address: '',
-                    amount: null,
+                    address: queryParams.address ? queryParams.address : '',
+                    amount: queryParams.amount ? queryParams.amount : null,
                     coinSymbol: coinList && coinList.length ? coinList[0].coin : '',
                     feeCoinSymbol: '',
                     message: '',
@@ -52,7 +58,7 @@
                 },
                 formAdvanced: {
                     feeCoinSymbol: '',
-                    message: '',
+                    message: queryParams.message ? queryParams.message : '',
                 },
                 isModeAdvanced: false,
                 isConfirmModalVisible: false,
shrpne commented 5 years ago

Спасибо за предложение, но пока что принято решение не внедрять это по соображениям безопасности использования.

dadon commented 5 years ago

В чем конкретно проблема с безопасностью у этого решения? Фича нужная, может обсудим как сделать более оптимально?

shrpne commented 5 years ago

Проблема в том, что нет прямой связи между переходом по ссылке и появлением значений в полях формы, это может сбивать с толку пользователя или он может не заметить значение в поле и отправить неправильную транзакцию и потерять средства.

Возможно, лучшим решением будет автозаполнение всех полей на основе считывания одного qr кода, будем еще обсуждать этот вопрос с командой.