minor fixes

apps/web/src/components/report/ReportDateRange.tsx

@@ -3,6 +3,8 @@ import { RadioGroup, RadioGroupItem } from "../ui/radio-group";
 import { changeDateRanges, changeInterval } from "./reportSlice";
 import { Combobox } from "../ui/combobox";
 import { type IInterval } from "@/types";
+import { timeRanges } from "@/utils/constants";
+import { entries } from "@/utils/object";
 
 export function ReportDateRange() {
   const dispatch = useDispatch();
@@ -12,62 +14,17 @@ export function ReportDateRange() {
   return (
     <>
       <RadioGroup>
-        <RadioGroupItem
-          onClick={() => {
-            dispatch(changeDateRanges("today"));
-          }}
-        >
-          Today
-        </RadioGroupItem>
-        <RadioGroupItem
-          onClick={() => {
-            dispatch(changeDateRanges(1));
-          }}
-        >
-          24 hours
-        </RadioGroupItem>
-        <RadioGroupItem
-          onClick={() => {
-            dispatch(changeDateRanges(7));
-          }}
-        >
-          7 days
-        </RadioGroupItem>
-        <RadioGroupItem
-          onClick={() => {
-            dispatch(changeDateRanges(14));
-          }}
-        >
-          14 days
-        </RadioGroupItem>
-        <RadioGroupItem
-          onClick={() => {
-            dispatch(changeDateRanges(30));
-          }}
-        >
-          1 month
-        </RadioGroupItem>
-        <RadioGroupItem
-          onClick={() => {
-            dispatch(changeDateRanges(90));
-          }}
-        >
-          3 month
-        </RadioGroupItem>
-        <RadioGroupItem
-          onClick={() => {
-            dispatch(changeDateRanges(180));
-          }}
-        >
-          6 month
-        </RadioGroupItem>
-        <RadioGroupItem
-          onClick={() => {
-            dispatch(changeDateRanges(356));
-          }}
-        >
-          1 year
-        </RadioGroupItem>
+        {entries(timeRanges).map(([range, title]) => (
+          <RadioGroupItem
+            key={range}
+            // active={range === interval}
+            onClick={() => {
+              dispatch(changeDateRanges(range));
+            }}
+          >
+            {title}
+          </RadioGroupItem>
+        ))}
       </RadioGroup>
       {chartType === "linear" && (
         <div className="w-full max-w-[200px]">

apps/web/src/components/report/sidebar/ReportSaveButton.tsx

@@ -1,16 +1,25 @@
 import { Button } from "@/components/ui/button";
 import { useReportId } from "../hooks/useReportId";
-import { api } from "@/utils/api";
+import { api, handleError } from "@/utils/api";
 import { useSelector } from "@/redux";
 import { pushModal } from "@/modals";
+import { toast } from "@/components/ui/use-toast";
 
 export function ReportSaveButton() {
   const { reportId } = useReportId();
-  const update = api.report.update.useMutation();
+  const update = api.report.update.useMutation({
+    onSuccess() {
+      toast({
+        title: "Success",
+        description: "Report updated.",
+      });
+    },
+    onError: handleError
+  });
   const report = useSelector((state) => state.report);
 
   if (reportId) {
-    return <Button onClick={() => {
+    return <Button loading={update.isLoading} onClick={() => {
       update.mutate({
         reportId,
         report,

apps/web/src/components/ui/button.tsx

@@ -3,6 +3,7 @@ import { Slot } from "@radix-ui/react-slot";
 import { cva, type VariantProps } from "class-variance-authority";
 
 import { cn } from "@/utils/cn";
+import { Loader2 } from "lucide-react";
 
 const buttonVariants = cva(
   "inline-flex items-center justify-center rounded-md text-sm font-medium ring-offset-background transition-colors focus-visible:outline-none focus-visible:ring-2 focus-visible:ring-ring focus-visible:ring-offset-2 disabled:pointer-events-none disabled:opacity-50",
@@ -37,18 +38,24 @@ interface ButtonProps
   extends React.ButtonHTMLAttributes<HTMLButtonElement>,
     VariantProps<typeof buttonVariants> {
   asChild?: boolean;
+  loading?: boolean;
 }
 
 const Button = React.forwardRef<HTMLButtonElement, ButtonProps>(
-  ({ className, variant, size, asChild = false, ...props }, ref) => {
+  (
+    { className, variant, size, asChild = false, children, loading, disabled, ...props },
+    ref,
+  ) => {
     const Comp = asChild ? Slot : "button";
     return (
-      // @ts-expect-error
       <Comp
         className={cn(buttonVariants({ variant, size, className }))}
         ref={ref}
+        disabled={loading ?? disabled}
         {...props}
-      />
+      >
+        {loading ? <Loader2 className="animate-spin" /> : <>{children}</>}
+      </Comp>
     );
   },
 );

apps/web/src/components/ui/radio-group.tsx

@@ -3,7 +3,9 @@ import * as React from "react"
 import { cn } from "@/utils/cn"
 
 export type RadioGroupProps = React.InputHTMLAttributes<HTMLDivElement>
-export type RadioGroupItemProps = React.InputHTMLAttributes<HTMLButtonElement>
+export type RadioGroupItemProps = React.InputHTMLAttributes<HTMLButtonElement> & {
+  active?: boolean
+}
 
 const RadioGroup = React.forwardRef<HTMLDivElement, RadioGroupProps>(
   ({ className, type, ...props }, ref) => {
@@ -20,9 +22,9 @@ const RadioGroup = React.forwardRef<HTMLDivElement, RadioGroupProps>(
   }
 )
 
-const RadioGroupItem = React.forwardRef<HTMLButtonElement, RadioGroupItemProps>(({className, ...props}, ref) => {
+const RadioGroupItem = React.forwardRef<HTMLButtonElement, RadioGroupItemProps>(({className, active, ...props}, ref) => {
   return (
-    <button {...props} className={cn('flex-1 px-3 whitespace-nowrap leading-none hover:bg-slate-100 transition-colors font-medium', className)} type="button" ref={ref} />
+    <button {...props} className={cn('flex-1 px-3 whitespace-nowrap leading-none hover:bg-slate-100 transition-colors font-medium', className, active && 'bg-slate-100')} type="button" ref={ref} />
   )
 })