From f2026bb921761ff9793dd6cfa11d2382f898032c Mon Sep 17 00:00:00 2001 From: Matthew Raymer Date: Mon, 18 Aug 2025 07:44:26 +0000 Subject: [PATCH] chore: clean up debug logging --- src/services/ProfileService.ts | 14 ++++---------- src/views/AccountViewView.vue | 20 -------------------- 2 files changed, 4 insertions(+), 30 deletions(-) diff --git a/src/services/ProfileService.ts b/src/services/ProfileService.ts index bdb27f46..f26b7052 100644 --- a/src/services/ProfileService.ts +++ b/src/services/ProfileService.ts @@ -124,17 +124,12 @@ export class ProfileService { async deleteProfile(activeDid: string): Promise { try { const headers = await getHeaders(activeDid); - logger.debug("Attempting to delete profile for DID:", activeDid); - logger.debug("Using partner API server:", this.partnerApiServer); - logger.debug("Request headers:", headers); - - const url = `${this.partnerApiServer}/api/partner/userProfile`; - logger.debug("DELETE request URL:", url); - - const response = await this.axios.delete(url, { headers }); + const response = await this.axios.delete( + `${this.partnerApiServer}/api/partner/userProfile`, + { headers }, + ); if (response.status === 200 || response.status === 204) { - logger.debug("Profile deleted successfully"); return true; } else { logger.error("Unexpected response status when deleting profile:", { @@ -156,7 +151,6 @@ export class ProfileService { // Handle specific HTTP status codes if (response.status === 204) { - logger.debug("Profile deleted successfully (204 No Content)"); return true; // 204 is success for DELETE operations } else if (response.status === 404) { logger.warn("Profile not found - may already be deleted"); diff --git a/src/views/AccountViewView.vue b/src/views/AccountViewView.vue index 1c38a8bb..ceffa5cd 100644 --- a/src/views/AccountViewView.vue +++ b/src/views/AccountViewView.vue @@ -182,7 +182,6 @@ @change="onLocationCheckboxChange" /> - (Debug: {{ isMapReady ? 'Map Ready' : 'Map Loading' }})

@@ -1541,14 +1540,12 @@ export default class AccountViewView extends Vue { onMapReady(map: L.Map): void { try { - logger.debug("Map ready event fired, map object:", map); // doing this here instead of on the l-map element avoids a recentering after a drag then zoom at startup const zoom = this.userProfileLatitude && this.userProfileLongitude ? 12 : 2; const lat = this.userProfileLatitude || 0; const lng = this.userProfileLongitude || 0; map.setView([lat, lng], zoom); this.isMapReady = true; - logger.debug("Map ready state set to true, coordinates:", [lat, lng], "zoom:", zoom); } catch (error) { logger.error("Error in onMapReady:", error); this.isMapReady = true; // Set to true even on error to prevent infinite loading @@ -1556,27 +1553,18 @@ export default class AccountViewView extends Vue { } onMapMounted(): void { - logger.debug("Map component mounted"); - // Check if map ref is available - const mapRef = this.$refs.profileMap; - logger.debug("Map ref:", mapRef); - // Try to set map ready after component is mounted setTimeout(() => { this.isMapReady = true; - logger.debug("Map ready set to true after mounted"); }, 500); } // Fallback method to handle map initialization failures private handleMapInitFailure(): void { - logger.debug("Starting map initialization timeout (5 seconds)"); setTimeout(() => { if (!this.isMapReady) { logger.warn("Map failed to initialize, forcing ready state"); this.isMapReady = true; - } else { - logger.debug("Map initialized successfully, timeout not needed"); } }, 5000); // 5 second timeout } @@ -1598,8 +1586,6 @@ export default class AccountViewView extends Vue { includeLocation: this.includeUserProfileLocation, }; - logger.debug("Saving profile data:", profileData); - const success = await this.profileService.saveProfile( this.activeDid, profileData, @@ -1664,7 +1650,6 @@ export default class AccountViewView extends Vue { async deleteProfile(): Promise { try { - logger.debug("Attempting to delete profile for DID:", this.activeDid); const success = await this.profileService.deleteProfile(this.activeDid); if (success) { this.notify.success(ACCOUNT_VIEW_CONSTANTS.SUCCESS.PROFILE_DELETED); @@ -1673,7 +1658,6 @@ export default class AccountViewView extends Vue { this.userProfileLongitude = 0; this.includeUserProfileLocation = false; this.isMapReady = false; // Reset map state - logger.debug("Profile deleted successfully, UI state reset"); } else { this.notify.error(ACCOUNT_VIEW_CONSTANTS.ERRORS.PROFILE_DELETE_ERROR); } @@ -1710,22 +1694,18 @@ export default class AccountViewView extends Vue { onLocationCheckboxChange(): void { try { - logger.debug("Location checkbox changed, new value:", this.includeUserProfileLocation); if (!this.includeUserProfileLocation) { // Location checkbox was unchecked, clean up map state this.isMapReady = false; this.userProfileLatitude = 0; this.userProfileLongitude = 0; - logger.debug("Location unchecked, map state reset"); } else { // Location checkbox was checked, start map initialization timeout this.isMapReady = false; - logger.debug("Location checked, starting map initialization timeout"); // Try to set map ready after a short delay to allow Vue to render setTimeout(() => { if (!this.isMapReady) { - logger.debug("Setting map ready after timeout"); this.isMapReady = true; } }, 1000); // 1 second delay