-
Notifications
You must be signed in to change notification settings - Fork 2
Removed console logs and fixed small privy login error #32
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
base: main
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -201,7 +201,6 @@ export default function AdminPage() { | |
| const res = await authFetch(url) | ||
|
|
||
| const e = await res.json() | ||
| console.log(e) | ||
| setEvents(e) | ||
| } | ||
| catch { | ||
|
|
@@ -342,7 +341,6 @@ export default function AdminPage() { | |
| setIsProcessing(true) | ||
| try { | ||
| //Token conversion call goes here | ||
| console.log("Token Conversion") | ||
| } catch (error) { | ||
| toast({ | ||
| title: "Conversion Failed", | ||
|
|
@@ -407,7 +405,6 @@ export default function AdminPage() { | |
| setIsConvertingToPaypal(true) | ||
| try { | ||
| // PayPal offload called here | ||
| console.log("PayPal offload") | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Same as above - is this a place holder for future work? If so prefix with comment with |
||
| } catch (error) { | ||
| toast({ | ||
| title: "PayPal Conversion Failed", | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -18,7 +18,6 @@ export default function PlaceAutocomplete({ setGoogleSubLocation, setBusinessPho | |
| if (!containerRef.current) return; | ||
|
|
||
| const importGoogleLibrary = async () => { | ||
| console.log("google maps imported") | ||
| await google.maps.importLibrary("places") as google.maps.PlacesLibrary; | ||
| } | ||
|
|
||
|
|
@@ -44,7 +43,6 @@ export default function PlaceAutocomplete({ setGoogleSubLocation, setBusinessPho | |
|
|
||
| ] }); | ||
| const rawGoogleData = place.toJSON() | ||
| console.log(rawGoogleData) | ||
| const googleDetails: GoogleSubLocation = { | ||
| google_id: rawGoogleData.id, | ||
| name: rawGoogleData.displayName, | ||
|
|
@@ -62,7 +60,6 @@ export default function PlaceAutocomplete({ setGoogleSubLocation, setBusinessPho | |
| maps_page: rawGoogleData.googleMapsURI, | ||
| opening_hours: rawGoogleData.regularOpeningHours?.weekdayDescriptions || [], | ||
| } | ||
| console.log(googleDetails) | ||
| setGoogleSubLocation(googleDetails) | ||
| if (typeof googleDetails.phone === "string") { | ||
| setBusinessPhone(googleDetails.phone) | ||
|
|
@@ -78,7 +75,6 @@ export default function PlaceAutocomplete({ setGoogleSubLocation, setBusinessPho | |
| placeAutocomplete.className="text-black dark:text-white border rounded-md bg-secondary px-3 py-2" | ||
|
|
||
| if (containerRef.current?.querySelector("gmp-place-autocomplete")) { | ||
| console.log("Element is already inside container"); | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Strange to leave the positive case here; should just have the single negative case. |
||
| } else { | ||
| //@ts-ignore | ||
| containerRef.current.appendChild(placeAutocomplete) | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -188,7 +188,6 @@ function useToast() { | |
|
|
||
| const toastFunction = React.useCallback(({ title, description, variant = "default" }: ToastProps) => { | ||
| // Mock toast implementation | ||
| console.log(`Toast: ${title} - ${description} (${variant})`) | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Prob makes sense to leave the |
||
| }, []) | ||
|
|
||
| return { | ||
|
|
||
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.
Is this a placeholder for future work? Otherwise, doesn't make sense to leave the
try.