From b3de9779b5a739c7887a6f86fd1796ee53f0937c Mon Sep 17 00:00:00 2001
From: Szymon Lesisz <sz.lesisz@gmail.com>
Date: Wed, 27 Feb 2019 18:05:54 +0100
Subject: [PATCH 1/2] disable "destination tag" in sendForm

---
 .../ripple/components/AdvancedForm/index.js   | 28 +++++++++----------
 1 file changed, 14 insertions(+), 14 deletions(-)

diff --git a/src/views/Wallet/views/Account/Send/ripple/components/AdvancedForm/index.js b/src/views/Wallet/views/Account/Send/ripple/components/AdvancedForm/index.js
index 7257b3be..873f9b2b 100644
--- a/src/views/Wallet/views/Account/Send/ripple/components/AdvancedForm/index.js
+++ b/src/views/Wallet/views/Account/Send/ripple/components/AdvancedForm/index.js
@@ -79,16 +79,16 @@ const getFeeInputState = (feeErrors: string, feeWarnings: string): string => {
     return state;
 };
 
-const getDestinationTagInputState = (errors: string, warnings: string): string => {
-    let state = '';
-    if (warnings && !errors) {
-        state = 'warning';
-    }
-    if (errors) {
-        state = 'error';
-    }
-    return state;
-};
+// const getDestinationTagInputState = (errors: string, warnings: string): string => {
+//     let state = '';
+//     if (warnings && !errors) {
+//         state = 'warning';
+//     }
+//     if (errors) {
+//         state = 'error';
+//     }
+//     return state;
+// };
 
 const Left = styled.div`
     display: flex;
@@ -106,11 +106,11 @@ const AdvancedForm = (props: Props) => {
         warnings,
         infos,
         fee,
-        destinationTag,
+        // destinationTag,
     } = props.sendForm;
     const {
         onFeeChange,
-        onDestinationTagChange,
+        // onDestinationTagChange,
     } = props.sendFormActions;
 
     return (
@@ -151,7 +151,7 @@ const AdvancedForm = (props: Props) => {
                 />
             </InputRow>
 
-            <InputRow>
+            {/* <InputRow>
                 <StyledInput
                     state={getDestinationTagInputState(errors.destinationTag, warnings.destinationTag)}
                     autoComplete="off"
@@ -185,7 +185,7 @@ const AdvancedForm = (props: Props) => {
                     value={destinationTag}
                     onChange={event => onDestinationTagChange(event.target.value)}
                 />
-            </InputRow>
+            </InputRow> */}
 
             <AdvancedSettingsSendButtonWrapper>
                 { props.children }

From 226685f7fe98a81d718eeece93fb9986251c4712 Mon Sep 17 00:00:00 2001
From: Szymon Lesisz <sz.lesisz@gmail.com>
Date: Wed, 27 Feb 2019 18:06:15 +0100
Subject: [PATCH 2/2] add XRP destination_tag notification to sendForm

---
 .../Context/components/Static/index.js            | 15 +++++++++++++++
 1 file changed, 15 insertions(+)

diff --git a/src/components/notifications/Context/components/Static/index.js b/src/components/notifications/Context/components/Static/index.js
index a6867a0e..c225a19e 100644
--- a/src/components/notifications/Context/components/Static/index.js
+++ b/src/components/notifications/Context/components/Static/index.js
@@ -33,6 +33,21 @@ export default (props: Props) => {
                     )}
                 />,
             );
+        } else if (location.state.send) {
+            notifications.push(
+                <Notification
+                    key="xrp-warning"
+                    type="warning"
+                    title="Do not send to accounts requiring a destination tag!"
+                    message={(
+                        <>
+                            Destination tag is an arbitrary number which serves as a unique identifier of your transaction. Some services may require this to process your transaction. The current firmware version <strong>does not support</strong> destination tags yet.
+                            <br /><br />
+                            If the receiver requires a destination tag, do not use Trezor to send XRP. We are working on adding this feature.
+                        </>
+                    )}
+                />,
+            );
         }
     }