Rewards code review

src/App.tsx

@@ -22,6 +22,8 @@ function App() {
 
 	const queryClient = new QueryClient()
 
+	// todo: we don't need SelectionProvider and DataProvider in app.concero.io anymore, these have to be removed.
+	// todo: the dark mode should be implemented via themeProvider later
 	return (
 		<PostHogProvider>
 			<I18Provider>

src/components/modals/RewardsUserHistory/UserAction.tsx

@@ -26,6 +26,7 @@ export const UserAction = ({ action }: UserActionProps) => {
 	}
 
 	const getQuestInfo = () => {
+		// todo: can we remove all "as" type casts?
 		const { name } = action.data as UserActionQuestData
 		setValue(`Quest completed: ${name}`)
 	}

src/components/rewards/Quests/QuestsGroup/QuestsGroup.tsx

@@ -19,15 +19,18 @@ export const QuestsGroup = ({ user }: QuestsCardProps) => {
 	const handleGetQuests = async () => {
 		let userQuestActions: IUserAction[] = []
 
+		// todo: calling fetchQuests twice
 		if (!user && !address) {
 			setQuests(await fetchQuests())
 		}
 
 		if (!user) return
 
+		// todo: promise.all
 		userQuestActions = await fetchUserQuestActions(user.address)
 		const fetchedQuests = await fetchQuests()
 
+		// todo: what is this used for?
 		const newQuests = fetchedQuests.map(quest => {
 			const userAction = userQuestActions.find(action => {
 				return action.documentId.toLocaleLowerCase() === quest._id.toLocaleLowerCase()
@@ -39,14 +42,17 @@ export const QuestsGroup = ({ user }: QuestsCardProps) => {
 	}
 
 	useEffect(() => {
+		// todo: this should not be named a handle as its not connected to a button
 		void handleGetQuests()
 	}, [user])
 
+	// todo: let's send sorted data from BE and not use filters and finds on the frontend
 	const campaignQuest = quests.find((quest: IQuest) => quest.type === QuestType.Campaign)
 	const dailyQuests = quests.filter((quest: IQuest) => quest.type === QuestType.Daily)
 	const primaryQuests = quests.filter((quest: IQuest) => quest.type === QuestType.Primary)
 	const secondaryQuests = quests.filter((quest: IQuest) => quest.type === QuestType.Secondary)
 
+	// todo: we have dailyQuests, therefore quests should be renamed
 	return (
 		<div className="gap-xxl">
 			{campaignQuest && (

src/components/screens/RewardsScreen/RewardsScreen.module.pcss

@@ -4,7 +4,7 @@
 	align-items: center;
 	margin: var(--sp-lg);
 }
-
+/*todo: rename to rewardsWrapper*/
 .rewardsWrap {
 	max-width: 600px;
 	width: 100%;

src/components/screens/RewardsScreen/RewardsScreen.tsx

@@ -35,6 +35,7 @@ export const RewardsScreen = () => {
 		setUser(currentUser!)
 	}
 
+	// todo: refactor user part into a separate component
 	return (
 		<>
 			<div className={classNames.rewardsScreenContainer}>