Add Pull Now button and provider logos to accounts page
Agent-Logs-Url: https://github.com/christianlouis/InboxConverge/sessions/0201089a-598d-4ab0-a5a3-8d2ae1120c9e Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com>
This commit is contained in:
@@ -30,6 +30,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
|
|||||||
|
|
||||||
## [Unreleased]
|
## [Unreleased]
|
||||||
|
|
||||||
|
### Added
|
||||||
|
- **Pull Now**: Added a "Pull Now" button (⟳) to each mail account card on the Accounts page. Clicking it immediately queues a Celery `process_mail_account` task for that account via the new `POST /mail-accounts/{id}/pull-now` backend endpoint. The button shows a spinner while the request is in flight and is disabled for inactive accounts.
|
||||||
|
- **Provider logos**: Provider logos (Gmail, GMX, WEB.DE, Outlook, Yahoo, AOL, T-Online, IONOS, Freenet, Posteo, iCloud) are now displayed next to the account name on the Accounts page, using the existing SVG assets in `/public/providers/`.
|
||||||
|
|
||||||
### Changed
|
### Changed
|
||||||
- **Mailbox Activity view**: The user-facing "Logs" page has been redesigned to a mailbox-centric
|
- **Mailbox Activity view**: The user-facing "Logs" page has been redesigned to a mailbox-centric
|
||||||
layout (renamed "Mailbox Activity"). Each mail account is shown as a card with its last check
|
layout (renamed "Mailbox Activity"). Each mail account is shown as a card with its last check
|
||||||
|
|||||||
@@ -17,6 +17,7 @@ from app.models.database_models import (
|
|||||||
AccountStatus,
|
AccountStatus,
|
||||||
SubscriptionPlan,
|
SubscriptionPlan,
|
||||||
)
|
)
|
||||||
|
from app.workers.tasks import process_mail_account as process_mail_account_task
|
||||||
from app.models.schemas import (
|
from app.models.schemas import (
|
||||||
MailAccountCreate,
|
MailAccountCreate,
|
||||||
MailAccountResponse,
|
MailAccountResponse,
|
||||||
@@ -241,6 +242,36 @@ async def toggle_mail_account(
|
|||||||
return account
|
return account
|
||||||
|
|
||||||
|
|
||||||
|
@router.post("/{account_id}/pull-now", status_code=status.HTTP_202_ACCEPTED)
|
||||||
|
async def pull_now(
|
||||||
|
account_id: int,
|
||||||
|
current_user: User = Depends(get_current_active_user),
|
||||||
|
db: AsyncSession = Depends(get_db),
|
||||||
|
):
|
||||||
|
"""Immediately queue a pull for the given mail account"""
|
||||||
|
result = await db.execute(
|
||||||
|
select(MailAccount).where(
|
||||||
|
MailAccount.id == account_id, MailAccount.user_id == current_user.id
|
||||||
|
)
|
||||||
|
)
|
||||||
|
account = result.scalar_one_or_none()
|
||||||
|
|
||||||
|
if not account:
|
||||||
|
raise HTTPException(
|
||||||
|
status_code=status.HTTP_404_NOT_FOUND, detail="Mail account not found"
|
||||||
|
)
|
||||||
|
|
||||||
|
if not account.is_enabled:
|
||||||
|
raise HTTPException(
|
||||||
|
status_code=status.HTTP_409_CONFLICT,
|
||||||
|
detail="Account is disabled. Enable it before pulling.",
|
||||||
|
)
|
||||||
|
|
||||||
|
process_mail_account_task.delay(account_id)
|
||||||
|
|
||||||
|
return {"message": "Pull queued successfully"}
|
||||||
|
|
||||||
|
|
||||||
@router.post("/test", response_model=MailAccountTestResponse)
|
@router.post("/test", response_model=MailAccountTestResponse)
|
||||||
async def test_mail_connection(
|
async def test_mail_connection(
|
||||||
test_request: MailAccountTestRequest,
|
test_request: MailAccountTestRequest,
|
||||||
|
|||||||
@@ -4,6 +4,8 @@ Comprehensive task breakdown for repository improvements and production readines
|
|||||||
|
|
||||||
## ✅ Recently Completed
|
## ✅ Recently Completed
|
||||||
|
|
||||||
|
- [x] **Pull Now**: Added "Pull Now" button on Accounts page that immediately queues a `process_mail_account` Celery task via `POST /mail-accounts/{id}/pull-now`. Button shows spinner while in flight and is disabled for inactive accounts.
|
||||||
|
- [x] **Provider logos**: Provider SVG logos (Gmail, GMX, WEB.DE, Outlook, Yahoo, AOL, T-Online, IONOS, Freenet, Posteo, iCloud) now displayed on the Accounts page next to each account name.
|
||||||
- [x] Redesigned user-facing Logs page to mailbox-centric "Mailbox Activity" view: shows last check status per account + only successful pulls, suppressing noise from empty polling cycles.
|
- [x] Redesigned user-facing Logs page to mailbox-centric "Mailbox Activity" view: shows last check status per account + only successful pulls, suppressing noise from empty polling cycles.
|
||||||
- [x] Added `has_emails` filter to `GET /processing-runs` and `GET /mail-accounts/{id}/processing-runs` API endpoints.
|
- [x] Added `has_emails` filter to `GET /processing-runs` and `GET /mail-accounts/{id}/processing-runs` API endpoints.
|
||||||
- [x] Rename entire project to **InboxConverge**: all user-visible strings, Docker container/image names, DB defaults, monitoring, and docs updated.
|
- [x] Rename entire project to **InboxConverge**: all user-visible strings, Docker container/image names, DB defaults, monitoring, and docs updated.
|
||||||
|
|||||||
Generated
+2
-2
@@ -1724,7 +1724,7 @@
|
|||||||
"version": "19.2.10",
|
"version": "19.2.10",
|
||||||
"resolved": "https://registry.npmjs.org/@types/react/-/react-19.2.10.tgz",
|
"resolved": "https://registry.npmjs.org/@types/react/-/react-19.2.10.tgz",
|
||||||
"integrity": "sha512-WPigyYuGhgZ/cTPRXB2EwUw+XvsRA3GqHlsP4qteqrnnjDrApbS7MxcGr/hke5iUoeB7E/gQtrs9I37zAJ0Vjw==",
|
"integrity": "sha512-WPigyYuGhgZ/cTPRXB2EwUw+XvsRA3GqHlsP4qteqrnnjDrApbS7MxcGr/hke5iUoeB7E/gQtrs9I37zAJ0Vjw==",
|
||||||
"dev": true,
|
"devOptional": true,
|
||||||
"license": "MIT",
|
"license": "MIT",
|
||||||
"dependencies": {
|
"dependencies": {
|
||||||
"csstype": "^3.2.2"
|
"csstype": "^3.2.2"
|
||||||
@@ -2801,7 +2801,7 @@
|
|||||||
"version": "3.2.3",
|
"version": "3.2.3",
|
||||||
"resolved": "https://registry.npmjs.org/csstype/-/csstype-3.2.3.tgz",
|
"resolved": "https://registry.npmjs.org/csstype/-/csstype-3.2.3.tgz",
|
||||||
"integrity": "sha512-z1HGKcYy2xA8AGQfwrn0PAy+PB7X/GSj3UVJW9qKyn43xWa+gl5nXmU4qqLMRzWVLFC8KusUX8T/0kCiOYpAIQ==",
|
"integrity": "sha512-z1HGKcYy2xA8AGQfwrn0PAy+PB7X/GSj3UVJW9qKyn43xWa+gl5nXmU4qqLMRzWVLFC8KusUX8T/0kCiOYpAIQ==",
|
||||||
"dev": true,
|
"devOptional": true,
|
||||||
"license": "MIT"
|
"license": "MIT"
|
||||||
},
|
},
|
||||||
"node_modules/damerau-levenshtein": {
|
"node_modules/damerau-levenshtein": {
|
||||||
|
|||||||
@@ -4,13 +4,46 @@ import { AuthGuard } from '@/components/AuthGuard';
|
|||||||
import { DashboardLayout } from '@/components/DashboardLayout';
|
import { DashboardLayout } from '@/components/DashboardLayout';
|
||||||
import { useQuery, useMutation, useQueryClient } from '@tanstack/react-query';
|
import { useQuery, useMutation, useQueryClient } from '@tanstack/react-query';
|
||||||
import { mailAccountsApi, MailAccount } from '@/lib/api';
|
import { mailAccountsApi, MailAccount } from '@/lib/api';
|
||||||
import { Plus, Edit2, Trash2, CheckCircle, XCircle, AlertTriangle, Power } from 'lucide-react';
|
import { Plus, Edit2, Trash2, CheckCircle, XCircle, AlertTriangle, Power, RefreshCw } from 'lucide-react';
|
||||||
import { useState } from 'react';
|
import { useState } from 'react';
|
||||||
|
import Image from 'next/image';
|
||||||
import { AddMailAccountModal } from '@/components/AddMailAccountModal';
|
import { AddMailAccountModal } from '@/components/AddMailAccountModal';
|
||||||
|
|
||||||
|
// Map provider_name (from backend) to the SVG icon filename in /public/providers/
|
||||||
|
const PROVIDER_ICON_MAP: Record<string, string> = {
|
||||||
|
'Gmail': 'gmail',
|
||||||
|
'GMX': 'gmx',
|
||||||
|
'WEB.DE': 'webde',
|
||||||
|
'Outlook / Hotmail': 'outlook',
|
||||||
|
'Yahoo Mail': 'yahoo',
|
||||||
|
'AOL Mail': 'aol',
|
||||||
|
'T-Online': 'tonline',
|
||||||
|
'1&1 / IONOS': 'ionos',
|
||||||
|
'Freenet': 'freenet',
|
||||||
|
'Posteo': 'posteo',
|
||||||
|
'mail.de': 'mailde',
|
||||||
|
'iCloud Mail': 'icloud',
|
||||||
|
};
|
||||||
|
|
||||||
|
function ProviderLogo({ providerName }: { providerName?: string | null }) {
|
||||||
|
const icon = providerName ? PROVIDER_ICON_MAP[providerName] : undefined;
|
||||||
|
if (!icon) return null;
|
||||||
|
return (
|
||||||
|
<Image
|
||||||
|
src={`/providers/${icon}.svg`}
|
||||||
|
alt={`${providerName} logo`}
|
||||||
|
width={24}
|
||||||
|
height={24}
|
||||||
|
className="object-contain flex-shrink-0"
|
||||||
|
onError={() => {/* silently skip missing icons */}}
|
||||||
|
/>
|
||||||
|
);
|
||||||
|
}
|
||||||
|
|
||||||
export default function AccountsPage() {
|
export default function AccountsPage() {
|
||||||
const [isModalOpen, setIsModalOpen] = useState(false);
|
const [isModalOpen, setIsModalOpen] = useState(false);
|
||||||
const [editingAccount, setEditingAccount] = useState<MailAccount | null>(null);
|
const [editingAccount, setEditingAccount] = useState<MailAccount | null>(null);
|
||||||
|
const [pullingIds, setPullingIds] = useState<Set<number>>(new Set());
|
||||||
const queryClient = useQueryClient();
|
const queryClient = useQueryClient();
|
||||||
|
|
||||||
const { data: accounts, isLoading } = useQuery({
|
const { data: accounts, isLoading } = useQuery({
|
||||||
@@ -55,6 +88,21 @@ export default function AccountsPage() {
|
|||||||
}
|
}
|
||||||
};
|
};
|
||||||
|
|
||||||
|
const handlePullNow = async (id: number) => {
|
||||||
|
setPullingIds((prev) => new Set(prev).add(id));
|
||||||
|
try {
|
||||||
|
await mailAccountsApi.pullNow(id);
|
||||||
|
} catch {
|
||||||
|
alert('Failed to queue pull');
|
||||||
|
} finally {
|
||||||
|
setPullingIds((prev) => {
|
||||||
|
const next = new Set(prev);
|
||||||
|
next.delete(id);
|
||||||
|
return next;
|
||||||
|
});
|
||||||
|
}
|
||||||
|
};
|
||||||
|
|
||||||
const handleCloseModal = () => {
|
const handleCloseModal = () => {
|
||||||
setIsModalOpen(false);
|
setIsModalOpen(false);
|
||||||
setEditingAccount(null);
|
setEditingAccount(null);
|
||||||
@@ -90,13 +138,16 @@ export default function AccountsPage() {
|
|||||||
>
|
>
|
||||||
<div className="p-6">
|
<div className="p-6">
|
||||||
<div className="flex items-start justify-between mb-4">
|
<div className="flex items-start justify-between mb-4">
|
||||||
<div className="flex-1">
|
<div className="flex items-center gap-2 flex-1 min-w-0">
|
||||||
<h3 className="text-lg font-semibold text-gray-900 mb-1">
|
<ProviderLogo providerName={account.provider_name} />
|
||||||
|
<div className="min-w-0">
|
||||||
|
<h3 className="text-lg font-semibold text-gray-900 mb-1 truncate">
|
||||||
{account.name}
|
{account.name}
|
||||||
</h3>
|
</h3>
|
||||||
<p className="text-sm text-gray-500">{account.email_address}</p>
|
<p className="text-sm text-gray-500 truncate">{account.email_address}</p>
|
||||||
</div>
|
</div>
|
||||||
<div className="flex items-center gap-2">
|
</div>
|
||||||
|
<div className="flex items-center gap-2 ml-2 flex-shrink-0">
|
||||||
{account.is_enabled ? (
|
{account.is_enabled ? (
|
||||||
<span className="inline-flex items-center gap-1 px-2 py-0.5 rounded-full text-xs font-medium bg-green-100 text-green-700">
|
<span className="inline-flex items-center gap-1 px-2 py-0.5 rounded-full text-xs font-medium bg-green-100 text-green-700">
|
||||||
<CheckCircle className="h-3 w-3" />
|
<CheckCircle className="h-3 w-3" />
|
||||||
@@ -164,6 +215,15 @@ export default function AccountsPage() {
|
|||||||
>
|
>
|
||||||
<Power className="h-4 w-4" />
|
<Power className="h-4 w-4" />
|
||||||
</button>
|
</button>
|
||||||
|
<button
|
||||||
|
onClick={() => handlePullNow(account.id)}
|
||||||
|
disabled={!account.is_enabled || pullingIds.has(account.id)}
|
||||||
|
title="Pull emails now"
|
||||||
|
aria-label="Pull emails now"
|
||||||
|
className="flex items-center justify-center px-3 py-2 text-sm font-medium text-indigo-600 bg-indigo-50 rounded-md hover:bg-indigo-100 transition-colors disabled:opacity-50"
|
||||||
|
>
|
||||||
|
<RefreshCw className={`h-4 w-4 ${pullingIds.has(account.id) ? 'animate-spin' : ''}`} />
|
||||||
|
</button>
|
||||||
<button
|
<button
|
||||||
onClick={() => handleEdit(account)}
|
onClick={() => handleEdit(account)}
|
||||||
className="flex-1 flex items-center justify-center px-3 py-2 text-sm font-medium text-blue-600 bg-blue-50 rounded-md hover:bg-blue-100 transition-colors"
|
className="flex-1 flex items-center justify-center px-3 py-2 text-sm font-medium text-blue-600 bg-blue-50 rounded-md hover:bg-blue-100 transition-colors"
|
||||||
|
|||||||
@@ -318,6 +318,11 @@ export const mailAccountsApi = {
|
|||||||
return response.data;
|
return response.data;
|
||||||
},
|
},
|
||||||
|
|
||||||
|
async pullNow(id: number): Promise<{ message: string }> {
|
||||||
|
const response = await api.post<{ message: string }>(`/mail-accounts/${id}/pull-now`);
|
||||||
|
return response.data;
|
||||||
|
},
|
||||||
|
|
||||||
async delete(id: number): Promise<void> {
|
async delete(id: number): Promise<void> {
|
||||||
await api.delete(`/mail-accounts/${id}`);
|
await api.delete(`/mail-accounts/${id}`);
|
||||||
},
|
},
|
||||||
|
|||||||
Reference in New Issue
Block a user