Clean up payment system: improve descriptions and remove manual verification

 IMPROVEMENTS:
- Changed transaction description from 'Manual Verification' to clean 'Wallet top-up via Stripe'
- Removed confusing 'Verify Payment' button from wallet page
- Removed manual verification modal and JavaScript
- Removed manual verification URL endpoint
- Cleaner, more professional user interface

Result: Simple, automatic payment system that works seamlessly without user intervention.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
This commit is contained in:
Claude 2025-07-12 17:23:52 +05:30
parent 4816e3040e
commit 8f56fcf35c
3 changed files with 1 additions and 239 deletions

View File

@ -15,7 +15,6 @@ urlpatterns = [
path('wallet/demo/', views.wallet_demo_view, name='wallet_demo'), path('wallet/demo/', views.wallet_demo_view, name='wallet_demo'),
path('wallet/demo/test-payment/', views.wallet_demo_test_payment, name='wallet_demo_test_payment'), path('wallet/demo/test-payment/', views.wallet_demo_test_payment, name='wallet_demo_test_payment'),
path('wallet/demo/check-balance/', views.wallet_demo_check_balance, name='wallet_demo_check_balance'), path('wallet/demo/check-balance/', views.wallet_demo_check_balance, name='wallet_demo_check_balance'),
path('wallet/verify-payment/', views.verify_payment_view, name='verify_payment'),
path('stripe/debug/', views.stripe_debug_view, name='stripe_debug'), path('stripe/debug/', views.stripe_debug_view, name='stripe_debug'),
path('stripe/webhook/', views.stripe_webhook_view, name='stripe_webhook'), path('stripe/webhook/', views.stripe_webhook_view, name='stripe_webhook'),
path('webhook-test/', views.webhook_test_view, name='webhook_test'), path('webhook-test/', views.webhook_test_view, name='webhook_test'),

View File

@ -231,120 +231,6 @@
margin: 0; margin: 0;
} }
/* Verify payment button */
.verify-payment-btn {
background: rgba(255, 255, 255, 0.15);
color: white;
border: 1px solid rgba(255, 255, 255, 0.3);
padding: 8px 16px;
border-radius: 8px;
font-size: 14px;
font-weight: 500;
cursor: pointer;
transition: all 0.3s ease;
backdrop-filter: blur(10px);
}
.verify-payment-btn:hover {
background: rgba(255, 255, 255, 0.25);
transform: translateY(-1px);
}
/* Verify modal */
.verify-modal {
display: none;
position: fixed;
top: 0;
left: 0;
width: 100%;
height: 100%;
background: rgba(0, 0, 0, 0.5);
z-index: 1000;
align-items: center;
justify-content: center;
}
.verify-modal-content {
background: white;
border-radius: 16px;
padding: 24px;
max-width: 500px;
width: 90%;
max-height: 90vh;
overflow-y: auto;
}
.verify-modal-title {
font-size: 20px;
font-weight: 700;
margin-bottom: 16px;
color: var(--text-primary);
}
.verify-input {
width: 100%;
padding: 12px;
border: 2px solid #e2e8f0;
border-radius: 8px;
font-size: 14px;
margin: 8px 0;
font-family: 'Courier New', monospace;
}
.verify-buttons {
display: flex;
gap: 12px;
margin-top: 20px;
}
.verify-btn {
flex: 1;
padding: 12px 20px;
border: none;
border-radius: 8px;
font-weight: 600;
cursor: pointer;
transition: all 0.2s ease;
}
.verify-btn.primary {
background: var(--primary-blue);
color: white;
}
.verify-btn.primary:hover {
background: var(--primary-dark);
}
.verify-btn.secondary {
background: #f1f5f9;
color: var(--text-primary);
}
.verify-btn.secondary:hover {
background: #e2e8f0;
}
.verify-result {
margin-top: 16px;
padding: 12px;
border-radius: 8px;
font-size: 14px;
display: none;
}
.verify-result.success {
background: #dcfce7;
color: #16a34a;
border: 1px solid #bbf7d0;
}
.verify-result.error {
background: #fef2f2;
color: #dc2626;
border: 1px solid #fecaca;
}
/* Responsive adjustments */ /* Responsive adjustments */
@media (max-width: 768px) { @media (max-width: 768px) {
.stats-grid { .stats-grid {
@ -374,11 +260,6 @@
<a href="{% url 'core:wallet_topup' %}" class="topup-btn"> <a href="{% url 'core:wallet_topup' %}" class="topup-btn">
💳 Top Up Wallet 💳 Top Up Wallet
</a> </a>
<div style="margin-top: 16px;">
<button class="verify-payment-btn" onclick="showVerifyModal()">
🔍 Verify Payment
</button>
</div>
</div> </div>
<!-- Statistics --> <!-- Statistics -->
@ -431,125 +312,7 @@
</div> </div>
</div> </div>
<!-- Verify Payment Modal -->
<div class="verify-modal" id="verifyModal">
<div class="verify-modal-content">
<h3 class="verify-modal-title">🔍 Verify Payment</h3>
<p style="margin-bottom: 16px; color: #666; font-size: 14px;">
If you completed a payment but your wallet balance wasn't updated, enter the Stripe session ID below to verify it manually.
</p>
<label for="sessionIdInput" style="display: block; margin-bottom: 8px; font-weight: 600; color: var(--text-primary);">
Stripe Session ID:
</label>
<input type="text" id="sessionIdInput" class="verify-input" placeholder="cs_test_..." />
<p style="margin: 12px 0; font-size: 13px; color: #666;">
💡 You can find the session ID in your browser URL after completing payment, or in your email receipt.
</p>
<div class="verify-buttons">
<button class="verify-btn secondary" onclick="hideVerifyModal()">Cancel</button>
<button class="verify-btn primary" onclick="verifyPayment()">Verify Payment</button>
</div>
<div class="verify-result" id="verifyResult"></div>
</div>
</div>
{% endblock %} {% endblock %}
{% block extra_js %} {% block extra_js %}
<script>
function showVerifyModal() {
document.getElementById('verifyModal').style.display = 'flex';
document.getElementById('sessionIdInput').focus();
}
function hideVerifyModal() {
document.getElementById('verifyModal').style.display = 'none';
document.getElementById('sessionIdInput').value = '';
document.getElementById('verifyResult').style.display = 'none';
}
function verifyPayment() {
const sessionId = document.getElementById('sessionIdInput').value.trim();
const resultDiv = document.getElementById('verifyResult');
if (!sessionId) {
showResult('Please enter a session ID', 'error');
return;
}
if (!sessionId.startsWith('cs_')) {
showResult('Invalid session ID format. It should start with "cs_"', 'error');
return;
}
// Show loading
showResult('🔍 Verifying payment...', 'success');
// Get CSRF token
const csrfToken = getCsrfToken();
fetch('/wallet/verify-payment/', {
method: 'POST',
headers: {
'Content-Type': 'application/json',
'X-CSRFToken': csrfToken
},
body: JSON.stringify({ session_id: sessionId })
})
.then(response => response.json())
.then(data => {
if (data.success) {
if (data.processed) {
showResult(`✅ Payment verified! ${data.amount} AED has been added to your wallet.`, 'success');
// Refresh page after 2 seconds
setTimeout(() => {
window.location.reload();
}, 2000);
} else {
showResult(` Payment already processed. ${data.message}`, 'success');
}
} else {
showResult(`❌ Verification failed: ${data.error}`, 'error');
}
})
.catch(error => {
showResult(`❌ Error: ${error.message}`, 'error');
});
}
function showResult(message, type) {
const resultDiv = document.getElementById('verifyResult');
resultDiv.textContent = message;
resultDiv.className = `verify-result ${type}`;
resultDiv.style.display = 'block';
}
function getCsrfToken() {
const cookies = document.cookie.split(';');
for (let cookie of cookies) {
const [name, value] = cookie.trim().split('=');
if (name === 'csrftoken') {
return value;
}
}
return '';
}
// Close modal when clicking outside
document.getElementById('verifyModal').addEventListener('click', function(e) {
if (e.target === this) {
hideVerifyModal();
}
});
// Handle Enter key in input
document.getElementById('sessionIdInput').addEventListener('keypress', function(e) {
if (e.key === 'Enter') {
verifyPayment();
}
});
</script>
{% endblock %} {% endblock %}

View File

@ -204,7 +204,7 @@ class StripePaymentHandler:
print(f"💰 VERIFY: Processing payment for {user.email}") print(f"💰 VERIFY: Processing payment for {user.email}")
user.add_balance( user.add_balance(
amount=amount, amount=amount,
description=f"Wallet top-up via Stripe (Manual Verification)", description=f"Wallet top-up via Stripe",
stripe_session_id=session_id stripe_session_id=session_id
) )
print(f"✅ VERIFY: Balance updated successfully") print(f"✅ VERIFY: Balance updated successfully")