feat(tenancy): add delete button on table + add-to-store in edit modal

Table actions now show view + edit + delete (trash icon) for non-owner
members. Delete opens the existing remove-from-all-stores modal.

Edit modal enhanced with "Add to another store" section:
- Shows a dashed-border card with store dropdown + role dropdown + add button
- Only appears when the member is not yet in all merchant stores
- Uses the existing invite API to add the member to the selected store

i18n: 2 new keys (add_to_store, select_store) in 4 locales.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

app/modules/prospecting/schemas/score.py

@@ -18,7 +18,7 @@ class ProspectScoreResponse(BaseModel):
     business_value_score: int
     engagement_score: int
     reason_flags: list[str] = []
-    score_breakdown: dict[str, int] | None = None
+    score_breakdown: dict[str, dict[str, int]] | None = None
     lead_tier: str | None = None
     notes: str | None = None
     created_at: datetime

app/modules/prospecting/services/scoring_service.py

@@ -103,33 +103,38 @@ class ScoringService:
         business_value = 0
         engagement = 0
         flags = []
-        breakdown = {}
+        breakdown = {
+            "technical_health": {},
+            "modernity": {},
+            "business_value": {},
+            "engagement": {},
+        }
 
         # === TECHNICAL HEALTH (max 40) ===
         if not prospect.uses_https:
             tech_health += 15
             flags.append("no_ssl")
-            breakdown["no_ssl"] = 15
+            breakdown["technical_health"]["no_ssl"] = 15
 
         perf = prospect.performance_profile
         if perf and perf.performance_score is not None:
             if perf.performance_score < 30:
                 tech_health += 15
                 flags.append("very_slow")
-                breakdown["very_slow"] = 15
+                breakdown["technical_health"]["very_slow"] = 15
             elif perf.performance_score < 50:
                 tech_health += 10
                 flags.append("slow")
-                breakdown["slow"] = 10
+                breakdown["technical_health"]["slow"] = 10
             elif perf.performance_score < 70:
                 tech_health += 5
                 flags.append("moderate_speed")
-                breakdown["moderate_speed"] = 5
+                breakdown["technical_health"]["moderate_speed"] = 5
 
             if perf.is_mobile_friendly is False:
                 tech_health += 10
                 flags.append("not_mobile_friendly")
-                breakdown["not_mobile_friendly"] = 10
+                breakdown["technical_health"]["not_mobile_friendly"] = 10
 
         tech_health = min(tech_health, 40)
 
@@ -139,36 +144,36 @@ class ScoringService:
             if tp.cms and tp.cms.lower() in OUTDATED_CMS:
                 modernity += 15
                 flags.append("outdated_cms")
-                breakdown["outdated_cms"] = 15
+                breakdown["modernity"]["outdated_cms"] = 15
             elif tp.cms is None and prospect.has_website:
                 modernity += 5
                 flags.append("unknown_cms")
-                breakdown["unknown_cms"] = 5
+                breakdown["modernity"]["unknown_cms"] = 5
 
             if tp.js_framework and tp.js_framework.lower() == "jquery":
                 modernity += 5
                 flags.append("legacy_js")
-                breakdown["legacy_js"] = 5
+                breakdown["modernity"]["legacy_js"] = 5
 
             if not tp.analytics:
                 modernity += 5
                 flags.append("no_analytics")
-                breakdown["no_analytics"] = 5
+                breakdown["modernity"]["no_analytics"] = 5
 
         modernity = min(modernity, 25)
 
         # === BUSINESS VALUE (max 25) ===
         if prospect.has_website:
             business_value += 10
-            breakdown["has_website"] = 10
+            breakdown["business_value"]["has_website"] = 10
 
         if tp and tp.ecommerce_platform:
             business_value += 10
-            breakdown["has_ecommerce"] = 10
+            breakdown["business_value"]["has_ecommerce"] = 10
 
         if prospect.domain_name and len(prospect.domain_name) <= 15:
             business_value += 5
-            breakdown["short_domain"] = 5
+            breakdown["business_value"]["short_domain"] = 5
 
         business_value = min(business_value, 25)
 
@@ -177,7 +182,7 @@ class ScoringService:
         if contacts:
             engagement += 5
             flags.append("has_contacts")
-            breakdown["has_contacts"] = 5
+            breakdown["engagement"]["has_contacts"] = 5
 
         has_email = any(c.contact_type == "email" for c in contacts)
         has_phone = any(c.contact_type == "phone" for c in contacts)
@@ -185,11 +190,11 @@ class ScoringService:
         if has_email:
             engagement += 3
             flags.append("has_email")
-            breakdown["has_email"] = 3
+            breakdown["engagement"]["has_email"] = 3
         if has_phone:
             engagement += 2
             flags.append("has_phone")
-            breakdown["has_phone"] = 2
+            breakdown["engagement"]["has_phone"] = 2
 
         engagement = min(engagement, 10)
 
@@ -202,7 +207,12 @@ class ScoringService:
         business_value = 0
         engagement = 0
         flags = []
-        breakdown = {}
+        breakdown = {
+            "technical_health": {},
+            "modernity": {},
+            "business_value": {},
+            "engagement": {},
+        }
 
         # Offline prospects without a website are high opportunity
         if not prospect.has_website:
@@ -210,9 +220,9 @@ class ScoringService:
             modernity = 20
             business_value = 20
             flags.extend(["no_website"])
-            breakdown["no_website_tech"] = 30
-            breakdown["no_website_mod"] = 20
-            breakdown["no_website_biz"] = 20
+            breakdown["technical_health"]["no_website"] = 30
+            breakdown["modernity"]["no_website"] = 20
+            breakdown["business_value"]["no_website"] = 20
 
         # Check for gmail usage (from contacts)
         contacts = prospect.contacts or []
@@ -223,7 +233,7 @@ class ScoringService:
         if has_gmail:
             modernity += 10
             flags.append("uses_gmail")
-            breakdown["uses_gmail"] = 10
+            breakdown["modernity"]["uses_gmail"] = 10
 
         modernity = min(modernity, 25)
 
@@ -231,7 +241,7 @@ class ScoringService:
         if prospect.source in ("street", "networking_event", "referral"):
             engagement += 5
             flags.append("met_in_person")
-            breakdown["met_in_person"] = 5
+            breakdown["engagement"]["met_in_person"] = 5
 
         if contacts:
             has_email = any(c.contact_type == "email" for c in contacts)
@@ -239,11 +249,11 @@ class ScoringService:
             if has_email:
                 engagement += 3
                 flags.append("has_email")
-                breakdown["has_email"] = 3
+                breakdown["engagement"]["has_email"] = 3
             if has_phone:
                 engagement += 2
                 flags.append("has_phone")
-                breakdown["has_phone"] = 2
+                breakdown["engagement"]["has_phone"] = 2
 
         engagement = min(engagement, 10)
 

app/modules/prospecting/static/admin/js/prospect-detail.js

@@ -123,6 +123,19 @@ function prospectDetail(prospectId) {
             return 'text-gray-600';
         },
 
+        scoreCategories() {
+            var s = this.prospect?.score;
+            if (!s) return [];
+            var bd = s.score_breakdown || {};
+            var positive = ['has_website', 'has_contacts', 'has_email', 'has_phone', 'has_ssl', 'modern_cms', 'fast_site', 'mobile_friendly', 'has_ecommerce', 'short_domain', 'met_in_person'];
+            return [
+                { key: 'technical_health', label: 'Technical Health', score: s.technical_health_score, max: 40, flags: Object.entries(bd.technical_health || {}), positive: positive },
+                { key: 'modernity', label: 'Modernity', score: s.modernity_score, max: 25, flags: Object.entries(bd.modernity || {}), positive: positive },
+                { key: 'business_value', label: 'Business Value', score: s.business_value_score, max: 25, flags: Object.entries(bd.business_value || {}), positive: positive },
+                { key: 'engagement', label: 'Engagement', score: s.engagement_score, max: 10, flags: Object.entries(bd.engagement || {}), positive: positive },
+            ];
+        },
+
         techProfileEntries() {
             const tp = this.prospect?.tech_profile;
             if (!tp) return [];

app/modules/prospecting/templates/prospecting/admin/prospect-detail.html

@@ -93,32 +93,36 @@
         <div class="p-4 bg-white rounded-lg shadow-xs dark:bg-gray-800">
             {{ section_header('Score Breakdown', icon='chart-bar') }}
             <template x-if="prospect.score">
-                <div class="space-y-3">
-                    <div class="flex justify-between text-sm">
-                        <span class="text-gray-600 dark:text-gray-400">Technical Health</span>
-                        <span class="font-semibold text-gray-700 dark:text-gray-200" x-text="prospect.score.technical_health_score + '/40'"></span>
-                    </div>
-                    <div class="flex justify-between text-sm">
-                        <span class="text-gray-600 dark:text-gray-400">Modernity</span>
-                        <span class="font-semibold text-gray-700 dark:text-gray-200" x-text="prospect.score.modernity_score + '/25'"></span>
-                    </div>
-                    <div class="flex justify-between text-sm">
-                        <span class="text-gray-600 dark:text-gray-400">Business Value</span>
-                        <span class="font-semibold text-gray-700 dark:text-gray-200" x-text="prospect.score.business_value_score + '/25'"></span>
-                    </div>
-                    <div class="flex justify-between text-sm">
-                        <span class="text-gray-600 dark:text-gray-400">Engagement</span>
-                        <span class="font-semibold text-gray-700 dark:text-gray-200" x-text="prospect.score.engagement_score + '/10'"></span>
-                    </div>
-                    <div class="pt-3 border-t border-gray-100 dark:border-gray-700">
-                        <h4 class="text-xs font-semibold text-gray-500 dark:text-gray-400 uppercase mb-2">Issues</h4>
-                        <div class="flex flex-wrap gap-2">
-                            <template x-for="flag in prospect.score.reason_flags || []" :key="flag">
-                                <span class="px-2 py-1 text-xs font-medium bg-red-100 text-red-700 rounded-full dark:bg-red-900 dark:text-red-300"
-                                      x-text="flag.replace('_', ' ')"></span>
+                <div class="space-y-4">
+                    <template x-for="cat in scoreCategories()" :key="cat.key">
+                        <div>
+                            <div class="flex justify-between text-sm mb-1">
+                                <span class="font-medium text-gray-700 dark:text-gray-300" x-text="cat.label"></span>
+                                <span class="font-semibold text-gray-700 dark:text-gray-200" x-text="cat.score + '/' + cat.max"></span>
+                            </div>
+                            <!-- Progress bar -->
+                            <div class="w-full bg-gray-200 dark:bg-gray-700 rounded-full h-1.5 mb-2">
+                                <div class="h-1.5 rounded-full" :class="cat.score > 0 ? 'bg-purple-500' : 'bg-gray-300 dark:bg-gray-600'"
+                                     :style="'width:' + Math.round(cat.score / cat.max * 100) + '%'"></div>
+                            </div>
+                            <!-- Per-flag details -->
+                            <template x-if="cat.flags.length > 0">
+                                <div class="space-y-0.5 ml-2">
+                                    <template x-for="[flag, pts] in cat.flags" :key="flag">
+                                        <div class="flex items-center justify-between text-xs">
+                                            <span class="text-gray-500 dark:text-gray-400" x-text="flag.replaceAll('_', ' ')"></span>
+                                            <span class="font-semibold px-1.5 py-0.5 rounded"
+                                                  :class="cat.positive.includes(flag)
+                                                      ? 'text-green-700 bg-green-50 dark:text-green-300 dark:bg-green-900/30'
+                                                      : 'text-orange-700 bg-orange-50 dark:text-orange-300 dark:bg-orange-900/30'"
+                                                  x-text="pts + ' pts'"></span>
+                                        </div>
+                                    </template>
+                                </div>
                             </template>
+                            <p x-show="cat.flags.length === 0" class="text-xs text-gray-400 ml-2">No issues detected</p>
                         </div>
-                    </div>
+                    </template>
                 </div>
             </template>
             <p x-show="!prospect.score" class="text-sm text-gray-400 text-center py-4">Not scored yet</p>
@@ -127,6 +131,15 @@
         <!-- Tech Profile Summary -->
         <div x-show="prospect.tech_profile" class="p-4 bg-white rounded-lg shadow-xs dark:bg-gray-800">
             {{ section_header('Technology', icon='code') }}
+            <!-- CMS badge (prominent) -->
+            <div x-show="prospect.tech_profile?.cms" class="mb-3 flex items-center gap-2">
+                <span class="px-3 py-1.5 text-sm font-semibold rounded-lg bg-purple-100 text-purple-700 dark:bg-purple-900/40 dark:text-purple-300"
+                      x-text="(prospect.tech_profile?.cms || '').charAt(0).toUpperCase() + (prospect.tech_profile?.cms || '').slice(1) + (prospect.tech_profile?.cms_version ? ' ' + prospect.tech_profile.cms_version : '')"></span>
+                <span x-show="prospect.tech_profile?.ecommerce_platform"
+                      class="px-2 py-1 text-xs font-medium rounded-lg bg-blue-100 text-blue-700 dark:bg-blue-900/40 dark:text-blue-300"
+                      x-text="prospect.tech_profile?.ecommerce_platform"></span>
+            </div>
+            <p x-show="!prospect.tech_profile?.cms" class="mb-3 text-sm text-gray-500 dark:text-gray-400">Custom / Unknown CMS</p>
             <div class="space-y-2 text-sm">
                 <template x-for="[key, val] in techProfileEntries()" :key="key">
                     <div class="flex justify-between">
@@ -134,6 +147,15 @@
                         <span class="font-medium text-gray-700 dark:text-gray-300" x-text="val"></span>
                     </div>
                 </template>
+                <div x-show="prospect.tech_profile?.cert_issuer" class="flex justify-between">
+                    <span class="text-gray-600 dark:text-gray-400">SSL Issuer</span>
+                    <span class="font-medium text-gray-700 dark:text-gray-300" x-text="prospect.tech_profile?.cert_issuer"></span>
+                </div>
+                <div x-show="prospect.tech_profile?.cert_expires_at" class="flex justify-between">
+                    <span class="text-gray-600 dark:text-gray-400">SSL Expires</span>
+                    <span class="font-medium text-gray-700 dark:text-gray-300"
+                          x-text="prospect.tech_profile?.cert_expires_at ? new Date(prospect.tech_profile.cert_expires_at).toLocaleDateString() : '—'"></span>
+                </div>
             </div>
         </div>
 

app/modules/tenancy/locales/de.json

@@ -53,6 +53,8 @@
         "personal_info": "Persönliche Informationen",
         "resend_invitation": "Einladung erneut senden",
         "save_profile": "Profil speichern",
+        "add_to_store": "Zu einem anderen Shop hinzufügen",
+        "select_store": "Shop auswählen...",
         "multiple_roles": "Mehrere Rollen",
         "view_member": "Mitglied anzeigen",
         "account_information": "Kontoinformationen",

app/modules/tenancy/locales/en.json

@@ -53,6 +53,8 @@
         "personal_info": "Personal Information",
         "resend_invitation": "Resend Invitation",
         "save_profile": "Save Profile",
+        "add_to_store": "Add to another store",
+        "select_store": "Select store...",
         "multiple_roles": "Multiple roles",
         "view_member": "View Member",
         "account_information": "Account Information",

app/modules/tenancy/locales/fr.json

@@ -53,6 +53,8 @@
         "personal_info": "Informations personnelles",
         "resend_invitation": "Renvoyer l'invitation",
         "save_profile": "Enregistrer le profil",
+        "add_to_store": "Ajouter à un autre magasin",
+        "select_store": "Sélectionner un magasin...",
         "multiple_roles": "Rôles multiples",
         "view_member": "Voir le membre",
         "account_information": "Informations du compte",

app/modules/tenancy/locales/lb.json

@@ -53,6 +53,8 @@
         "personal_info": "Perséinlech Informatiounen",
         "resend_invitation": "Aluedung nei schécken",
         "save_profile": "Profil späicheren",
+        "add_to_store": "Zu engem anere Shop derbäisetzen",
+        "select_store": "Shop auswielen...",
         "multiple_roles": "Méi Rollen",
         "view_member": "Member kucken",
         "account_information": "Konto Informatiounen",

app/modules/tenancy/static/merchant/js/merchant-team.js

@@ -38,6 +38,12 @@ function merchantTeam() {
         showViewModal: false,
         selectedMember: null,
 
+        // Add to store form (inside edit modal)
+        addStoreForm: {
+            store_id: '',
+            role_name: 'staff',
+        },
+
         // Invite form
         inviteForm: {
             email: '',
@@ -186,6 +192,45 @@ function merchantTeam() {
             this.showInviteModal = true;
         },
 
+        /**
+         * Get stores the member is NOT yet in (for "Add to store" in edit modal)
+         */
+        getAvailableStores(member) {
+            if (!member || !member.stores) return this.stores;
+            const memberStoreIds = member.stores.map(s => s.store_id);
+            return this.stores.filter(s => !memberStoreIds.includes(s.id));
+        },
+
+        /**
+         * Add member to another store (invites them via existing invite API)
+         */
+        async addMemberToStore(member) {
+            if (!this.addStoreForm.store_id) return;
+
+            this.saving = true;
+            try {
+                await apiClient.post('/merchants/account/team/invite', {
+                    email: member.email,
+                    store_ids: [parseInt(this.addStoreForm.store_id)],
+                    role_name: this.addStoreForm.role_name,
+                });
+
+                Utils.showToast(I18n.t('tenancy.messages.invitation_sent_successfully'), 'success');
+                merchantTeamLog.info('Added member to store:', this.addStoreForm.store_id);
+
+                this.addStoreForm.store_id = '';
+                this.addStoreForm.role_name = 'staff';
+                this.showEditModal = false;
+                this.selectedMember = null;
+                await this.loadTeamData();
+            } catch (error) {
+                merchantTeamLog.error('Failed to add member to store:', error);
+                Utils.showToast(error.message || 'Failed to add member to store', 'error');
+            } finally {
+                this.saving = false;
+            }
+        },
+
         /**
          * Toggle store in invite form store_ids
          */
@@ -242,6 +287,7 @@ function merchantTeam() {
          */
         openEditModal(member) {
             this.selectedMember = JSON.parse(JSON.stringify(member));
+            this.addStoreForm = { store_id: '', role_name: 'staff' };
             this.showEditModal = true;
         },
 

app/modules/tenancy/templates/tenancy/merchant/team.html

@@ -137,11 +137,18 @@
                                     <span x-html="$icon('eye', 'w-4 h-4')"></span>
                                 </button>
                                 <template x-if="!member.is_owner">
-                                    <button @click="openEditModal(member)"
-                                            class="p-1.5 text-gray-500 hover:text-purple-600 dark:text-gray-400 dark:hover:text-purple-400 hover:bg-gray-100 dark:hover:bg-gray-700 rounded-lg transition-colors"
-                                            :title="$t('tenancy.team.edit_member')">
-                                        <span x-html="$icon('pencil', 'w-4 h-4')"></span>
-                                    </button>
+                                    <div class="flex items-center gap-2">
+                                        <button @click="openEditModal(member)"
+                                                class="p-1.5 text-gray-500 hover:text-purple-600 dark:text-gray-400 dark:hover:text-purple-400 hover:bg-gray-100 dark:hover:bg-gray-700 rounded-lg transition-colors"
+                                                :title="$t('tenancy.team.edit_member')">
+                                            <span x-html="$icon('pencil', 'w-4 h-4')"></span>
+                                        </button>
+                                        <button @click="openRemoveModal(member)"
+                                                class="p-1.5 text-gray-500 hover:text-red-600 dark:text-gray-400 dark:hover:text-red-400 hover:bg-gray-100 dark:hover:bg-gray-700 rounded-lg transition-colors"
+                                                :title="$t('tenancy.team.remove_member')">
+                                            <span x-html="$icon('trash', 'w-4 h-4')"></span>
+                                        </button>
+                                    </div>
                                 </template>
                                 <template x-if="member.is_owner">
                                     <span class="inline-flex items-center px-2 py-1 text-xs text-purple-600 dark:text-purple-400">
@@ -347,6 +354,34 @@
                 </div>
             </template>
 
+            <!-- Add to another store -->
+            <template x-if="getAvailableStores(selectedMember).length > 0">
+                <div class="p-3 border border-dashed border-gray-300 dark:border-gray-600 rounded-lg">
+                    <h5 class="text-xs font-medium text-gray-500 dark:text-gray-400 mb-2">{{ _('tenancy.team.add_to_store') }}</h5>
+                    <div class="flex items-center gap-2">
+                        <select x-model="addStoreForm.store_id"
+                                class="flex-1 px-2 py-1 text-sm text-gray-700 dark:text-gray-300 bg-white dark:bg-gray-700 border border-gray-300 dark:border-gray-600 rounded-md focus:border-purple-400 focus:outline-none">
+                            <option value="">{{ _('tenancy.team.select_store') }}</option>
+                            <template x-for="s in getAvailableStores(selectedMember)" :key="s.id">
+                                <option :value="s.id" x-text="s.name"></option>
+                            </template>
+                        </select>
+                        <select x-model="addStoreForm.role_name"
+                                class="px-2 py-1 text-sm text-gray-700 dark:text-gray-300 bg-white dark:bg-gray-700 border border-gray-300 dark:border-gray-600 rounded-md focus:border-purple-400 focus:outline-none">
+                            <template x-for="role in roleOptions" :key="role.value">
+                                <option :value="role.value" x-text="role.label"></option>
+                            </template>
+                        </select>
+                        <button @click="addMemberToStore(selectedMember)"
+                                :disabled="saving || !addStoreForm.store_id"
+                                class="px-3 py-1 text-xs font-medium text-white bg-purple-600 rounded-md hover:bg-purple-700 disabled:opacity-50 disabled:cursor-not-allowed transition-colors">
+                            <span x-show="saving" x-html="$icon('spinner', 'w-3 h-3')"></span>
+                            <span x-show="!saving" x-html="$icon('plus', 'w-3 h-3')"></span>
+                        </button>
+                    </div>
+                </div>
+            </template>
+
             <!-- Remove from all stores link -->
             <div x-show="selectedMember?.stores?.length > 1" class="text-center pt-2">
                 <button @click="showEditModal = false; openRemoveModal(selectedMember)"