shuashuai commented on code in PR #1192:
URL: https://github.com/apache/incubator-answer/pull/1192#discussion_r1872458207


##########
ui/src/components/Footer/index.tsx:
##########
@@ -33,6 +33,13 @@ const Index = () => {
     <footer className="bg-light">
       <Container className="py-3">
         <p className="text-center mb-0 small text-secondary">
+          <a className="m-2" href="/privacy" target="_blank">
+            Privacy Policy
+          </a>
+          <a href="/tos" target="_blank">
+            Terms of Service

Review Comment:
   > if u have the transaletd privacy policy and terms of service it will 
surely work out
   
   What I mean is that 'Terms of Services' should use i18n translation, not the 
content inside.
   
   eg:
   ```
   // import useTranslation
   import { Trans, useTranslation } from 'react-i18next';
   
   // Add translated content in tooter in `i18m/en_US.yaml`
   footer:
       build_on: >-
         Powered by <1> Apache Answer </1>- the open-source software that 
powers Q&A
         communities.<br />Made with love © {{cc}}.
       privacy: Privacy Policy
      terms_of_service: 
   
   // Use where translation is required on the page
   const { t } = useTranslation('translation', { keyPrefix: 'footer' });
   
   <a className="me-2" href="/privacy" target="_blank">
               {t('privacy')}
             </a>
             <a href="/tos" target="_blank">
               {t('terms_of_service')}
             </a>
   
   ```
   



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to