-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feat/stats #101
Feat/stats #101
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit
eslint
src/screens/profile/stadistics-screen.tsx|168 col 24| Inline style: { flex: 1, marginLeft: 5 }
src/screens/profile/test.tsx|1 col 1| Run autofix to sort these imports!
src/screens/profile/test.tsx|5 col 5| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|6 col 7| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|7 col 9| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|7 col 15| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|7 col 15| Classnames 'mt-4, mb-4' could be replaced by the 'my-4' shorthand!
src/screens/profile/test.tsx|8 col 11| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|8 col 17| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|9 col 13| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|10 col 15| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|11 col 17| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|11 col 23| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|11 col 23| Classname 'flex-grow' should be updated to 'grow' in Tailwind CSS v3!
src/screens/profile/test.tsx|12 col 19| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|12 col 25| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|16 col 19| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|16 col 25| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|20 col 17| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|20 col 23| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|21 col 19| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|21 col 25| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|26 col 15| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|26 col 21| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|27 col 17| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|27 col 23| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|28 col 19| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|28 col 19| Empty components are self-closing
src/screens/profile/test.tsx|30 col 17| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|36 col 9| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|36 col 15| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|37 col 11| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|37 col 17| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|38 col 13| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|39 col 15| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|40 col 17| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|40 col 23| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|40 col 23| Classname 'flex-grow' should be updated to 'grow' in Tailwind CSS v3!
src/screens/profile/test.tsx|41 col 19| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|41 col 25| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|44 col 19| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|44 col 25| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|48 col 17| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|48 col 23| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|49 col 19| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|49 col 25| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|54 col 15| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|54 col 21| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|55 col 17| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|55 col 23| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|56 col 19| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|56 col 19| Empty components are self-closing
src/screens/profile/test.tsx|58 col 17| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|64 col 9| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|64 col 15| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|65 col 11| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|65 col 17| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|66 col 13| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|67 col 15| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|68 col 17| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|68 col 23| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|68 col 23| Classname 'flex-grow' should be updated to 'grow' in Tailwind CSS v3!
src/screens/profile/test.tsx|69 col 19| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|69 col 25| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|72 col 19| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|72 col 25| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|76 col 17| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|76 col 23| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|77 col 19| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|77 col 25| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|82 col 15| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|82 col 21| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|83 col 17| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|83 col 23| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|84 col 19| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|84 col 19| Empty components are self-closing
src/screens/profile/test.tsx|86 col 17| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|92 col 9| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|92 col 15| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|93 col 11| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|93 col 17| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|94 col 13| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|95 col 15| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|96 col 17| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|96 col 23| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|96 col 23| Classname 'flex-grow' should be updated to 'grow' in Tailwind CSS v3!
src/screens/profile/test.tsx|97 col 19| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|97 col 25| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|100 col 19| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|100 col 25| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|104 col 17| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|104 col 23| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|105 col 19| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|105 col 25| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|110 col 15| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|110 col 21| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|111 col 17| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|111 col 23| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|112 col 19| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|112 col 19| Empty components are self-closing
src/screens/profile/test.tsx|114 col 17| 'React' must be in scope when using JSX
import React from 'react'; | ||
import { View, Text } from '@/ui'; | ||
|
||
import { FontAwesomeIcon } from '@fortawesome/react-native-fontawesome'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚫 [eslint] <simple-import-sort/imports> reported by reviewdog 🐶
Run autofix to sort these imports!
import React from 'react'; | |
import { View, Text } from '@/ui'; | |
import { FontAwesomeIcon } from '@fortawesome/react-native-fontawesome'; | |
import { FontAwesomeIcon } from '@fortawesome/react-native-fontawesome'; | |
import React from 'react'; | |
import { Text,View } from '@/ui'; |
icon: any; | ||
}) => { | ||
return ( | ||
<View className="mt-4 w-full lg:w-6/12 xl:w-3/12 px-5 mb-4 flex justify-end"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Invalid Tailwind CSS classnames order
<View className="mt-4 w-full lg:w-6/12 xl:w-3/12 px-5 mb-4 flex justify-end"> | |
<View className="mb-4 mt-4 flex w-full justify-end px-5 lg:w-6/12 xl:w-3/12"> |
icon: any; | ||
}) => { | ||
return ( | ||
<View className="mt-4 w-full lg:w-6/12 xl:w-3/12 px-5 mb-4 flex justify-end"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Classnames 'mt-4, mb-4' could be replaced by the 'my-4' shorthand!
<View className="mt-4 w-full lg:w-6/12 xl:w-3/12 px-5 mb-4 flex justify-end"> | |
<View className="my-4 w-full lg:w-6/12 xl:w-3/12 px-5 flex justify-end"> |
}) => { | ||
return ( | ||
<View className="mt-4 w-full lg:w-6/12 xl:w-3/12 px-5 mb-4 flex justify-end"> | ||
<View className="relative flex flex-col min-w-0 break-words bg-white rounded mb-3 xl:mb-0 shadow-lg"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Invalid Tailwind CSS classnames order
<View className="relative flex flex-col min-w-0 break-words bg-white rounded mb-3 xl:mb-0 shadow-lg"> | |
<View className="relative mb-3 flex min-w-0 flex-col break-words rounded bg-white shadow-lg xl:mb-0"> |
return ( | ||
<View className="mt-4 w-full lg:w-6/12 xl:w-3/12 px-5 mb-4 flex justify-end"> | ||
<View className="relative flex flex-col min-w-0 break-words bg-white rounded mb-3 xl:mb-0 shadow-lg"> | ||
<View className="flex-auto p-4 flex justify-between "> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Invalid Tailwind CSS classnames order
<View className="flex-auto p-4 flex justify-between "> | |
<View className="flex flex-auto justify-between p-4 "> |
)} | ||
<Text> | ||
Selected Start Date: {startDate.toISOString().split('T')[0]} | ||
<ScrollView style={{ flex: 1, padding: 10 }}> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Inline style: { flex: 1, padding: 10 }
<Text> | ||
Selected Start Date: {startDate.toISOString().split('T')[0]} | ||
<ScrollView style={{ flex: 1, padding: 10 }}> | ||
<View style={{ alignItems: 'center' }}> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Inline style: { alignItems: 'center' }
style={{ | ||
fontSize: 52, | ||
fontWeight: 'bold', | ||
textAlign: 'center', | ||
marginTop: 0, | ||
}} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Inline style: { fontSize: 52, fontWeight: 'bold', textAlign: 'center', marginTop: 0 }
style={{ | ||
flexDirection: 'row', | ||
justifyContent: 'space-between', | ||
marginVertical: 10, | ||
}} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Inline style: {
flexDirection: 'row',
justifyContent: 'space-between',
marginVertical: 10
}
marginVertical: 10, | ||
}} | ||
> | ||
<View style={{ flex: 1, marginRight: 5 }}> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Inline style: { flex: 1, marginRight: 5 }
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit
eslint
src/screens/profile/test.tsx|36 col 15| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|37 col 11| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|37 col 17| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|38 col 13| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|39 col 15| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|40 col 17| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|40 col 23| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|40 col 23| Classname 'flex-grow' should be updated to 'grow' in Tailwind CSS v3!
src/screens/profile/test.tsx|41 col 19| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|41 col 25| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|44 col 19| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|44 col 25| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|48 col 17| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|48 col 23| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|49 col 19| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|49 col 25| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|54 col 15| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|54 col 21| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|55 col 17| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|55 col 23| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|56 col 19| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|56 col 19| Empty components are self-closing
src/screens/profile/test.tsx|58 col 17| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|64 col 9| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|64 col 15| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|65 col 11| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|65 col 17| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|66 col 13| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|67 col 15| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|68 col 17| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|68 col 23| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|68 col 23| Classname 'flex-grow' should be updated to 'grow' in Tailwind CSS v3!
src/screens/profile/test.tsx|69 col 19| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|69 col 25| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|72 col 19| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|72 col 25| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|76 col 17| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|76 col 23| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|77 col 19| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|77 col 25| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|82 col 15| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|82 col 21| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|83 col 17| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|83 col 23| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|84 col 19| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|84 col 19| Empty components are self-closing
src/screens/profile/test.tsx|86 col 17| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|92 col 9| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|92 col 15| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|93 col 11| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|93 col 17| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|94 col 13| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|95 col 15| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|96 col 17| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|96 col 23| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|96 col 23| Classname 'flex-grow' should be updated to 'grow' in Tailwind CSS v3!
src/screens/profile/test.tsx|97 col 19| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|97 col 25| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|100 col 19| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|100 col 25| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|104 col 17| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|104 col 23| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|105 col 19| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|105 col 25| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|110 col 15| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|110 col 21| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|111 col 17| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|111 col 23| Invalid Tailwind CSS classnames order
src/screens/profile/test.tsx|112 col 19| 'React' must be in scope when using JSX
src/screens/profile/test.tsx|112 col 19| Empty components are self-closing
src/screens/profile/test.tsx|114 col 17| 'React' must be in scope when using JSX
)} | ||
</View> | ||
|
||
<View style={{ flex: 1, marginLeft: 5 }}> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Inline style: { flex: 1, marginLeft: 5 }
@@ -0,0 +1,124 @@ | |||
import { View, Text } from '@/ui'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚫 [eslint] <simple-import-sort/imports> reported by reviewdog 🐶
Run autofix to sort these imports!
import { View, Text } from '@/ui'; | |
import { Text,View } from '@/ui'; |
|
||
const Test = () => { | ||
return ( | ||
<View> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'React' must be in scope when using JSX
const Test = () => { | ||
return ( | ||
<View> | ||
<View className="flex flex-wrap bg-white "> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'React' must be in scope when using JSX
return ( | ||
<View> | ||
<View className="flex flex-wrap bg-white "> | ||
<View className="mt-4 w-full lg:w-6/12 xl:w-3/12 px-5 mb-4"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'React' must be in scope when using JSX
</View> | ||
</View> | ||
<Text className="text-sm text-blueGray-400 mt-4"> | ||
<Text className="text-emerald-500 mr-2"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Invalid Tailwind CSS classnames order
<Text className="text-emerald-500 mr-2"> | |
<Text className="mr-2 text-emerald-500"> |
</View> | ||
<Text className="text-sm text-blueGray-400 mt-4"> | ||
<Text className="text-emerald-500 mr-2"> | ||
<Text className="fas fa-arrow-up"></Text> 2,99%{' '} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'React' must be in scope when using JSX
</View> | ||
<Text className="text-sm text-blueGray-400 mt-4"> | ||
<Text className="text-emerald-500 mr-2"> | ||
<Text className="fas fa-arrow-up"></Text> 2,99%{' '} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Empty components are self-closing
<Text className="fas fa-arrow-up"></Text> 2,99%{' '} | |
<Text className="fas fa-arrow-up" /> 2,99%{' '} |
<Text className="text-emerald-500 mr-2"> | ||
<Text className="fas fa-arrow-up"></Text> 2,99%{' '} | ||
</Text> | ||
<Text className="whitespace-nowrap"> Since last month </Text> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'React' must be in scope when using JSX
</View> | ||
</View> | ||
|
||
<View className=" mt-4 w-full lg:w-6/12 xl:w-3/12 px-5"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'React' must be in scope when using JSX
¿Qué hace esto?
Estadisticas de Snaps
¿Por qué hiciste esto?
Feature
¿A quién/qué afecta esto?
End User