Apply saved filters to tender search with visible region criteria
This commit is contained in:
@@ -0,0 +1,15 @@
|
||||
{
|
||||
"date": "2026-09-06",
|
||||
"environment": "localhost:5182 five synthetic tenders",
|
||||
"checks": {
|
||||
"filteredOne": true,
|
||||
"regionVisible": true,
|
||||
"clearRestoresFive": true,
|
||||
"reapplyAfterReload": true
|
||||
},
|
||||
"limits": [
|
||||
"Region-only saved filter tested in browser",
|
||||
"Five criteria preservation and province coverage tested in unit suite",
|
||||
"No notification delivery, no real source accuracy proof"
|
||||
]
|
||||
}
|
||||
@@ -917,3 +917,10 @@ Taslak türü seçicisine erişilebilir ad eklendi. Üretim derlemesi geçti. Ö
|
||||
- Bildirim aboneliği oluşturmayan işlemdeki anında haber/bot çalıştırma iddiaları kaldırıldı. E-posta aboneliğinin ayrı kurulduğu açıklanır. Boş isim için alert yerine native required kullanılır; form alanları ve silme düğmeleri etiketlendi.
|
||||
- 206 test ve derleme geçti. Yerel tarayıcıda kaydetme onayı, yenilemede kayıt, silme ve yenilemede silinmiş kalma olmak üzere 4 kontrol geçti (qa-saved-filter-lifecycle.json). Bu tur kayıt hatası enjeksiyonu yapılmadı; çok cihazlı eşzamanlı düzenleme kabulü açık.
|
||||
- Ayrı eksik: kayıtlı filtre kartını arama ölçütlerine uygulama akışı mevcut değildi ve bu tur eklenmedi; bölge/şehir/sektör/usul ölçütleriyle uygulama sonraki iş olarak açık. Filtre yönetimi henüz tam uçtan uca kabul edilmiş sayılmaz.
|
||||
|
||||
### Kayıtlı filtreyi aramaya uygulama — 6 Eylül 2026
|
||||
- Kayıtlı filtre adları hem sol panelde hem filtre yönetiminde uygulanabilir düğme oldu. Uygulama şehir, bölge, sektör, kurum tipi ve usulü yükler; eski arama/CPV/hızlı filtreyi temizleyip ilan listesine geçer. Tanınmayan bölge geçersiz filtre hatası olarak gösterilir.
|
||||
- İl bazlı yedi bölge eşlemesi eklendi; 81 ilin her biri tam bir kez kapsanır. Aktif bölge listede açıkça görünür; tek başına veya tüm filtrelerle kaldırılabilir. Bölge şehir ölçütüyle birlikte uygulanır ve bilinmeyen şehir seçili bölgeye dahil edilmez.
|
||||
- Usul alanı explicit usul/ihaleUsulu/alimTuru değerlerinden eşleşir; Mal Alımı değerinden Açık İhale tahmini yapılmaz. Kaynakta usul yoksa usule göre filtrelenmiş sonuçta gösterilmez.
|
||||
- 209 otomatik test ve derleme geçti. 81 il kapsamı, Türkçe normalizasyon, beş ölçütün taşınması, geçersiz bölge ve usul/kategori ayrımı test edildi. Yerel 5 sentetik ihalede bölge filtresi 1 Giresun kaydına daralttı; bölgeyi kaldırınca 5 kayıt geri geldi; yenileme sonrası sol panelden tekrar uygulandı. 4 kontrol qa-saved-filter-application.json içinde.
|
||||
- İl bazında bölgeleme coğrafi sınırın ilçelerdeki ayrıntısını temsil etmez. Gerçek kaynak usul/şehir doğruluğu ve bütün ölçütlerin birleşik tarayıcı senaryosu ayrıca kabul edilmelidir. Bu işlem bildirim aboneliği oluşturmaz.
|
||||
|
||||
@@ -12,7 +12,7 @@ export const LeftSidebar = ({onOpenProductSearch} = {}) => {
|
||||
quickFilter,
|
||||
setQuickFilter,
|
||||
setActiveTab,
|
||||
savedFilters,
|
||||
savedFilters, applySavedFilter, selectedRegion,
|
||||
deleteSavedFilter,
|
||||
savingCount,
|
||||
selectedAlimTuru,
|
||||
@@ -89,7 +89,7 @@ export const LeftSidebar = ({onOpenProductSearch} = {}) => {
|
||||
<span className="font-bold text-xs uppercase tracking-wider text-slate-700 flex items-center gap-1.5">
|
||||
<Compass className="w-4 h-4 text-blue-600" /> Hızlı Erişim
|
||||
</span>
|
||||
{(quickFilter !== 'all' || selectedSektorler.length || selectedCpvler.length || selectedCity || selectedAlimTuru || selectedKurumTipi) && (
|
||||
{(selectedRegion || quickFilter !== 'all' || selectedSektorler.length || selectedCpvler.length || selectedCity || selectedAlimTuru || selectedKurumTipi) && (
|
||||
<button
|
||||
onClick={clearAllFilters}
|
||||
className="text-[11px] font-medium text-blue-600 hover:underline"
|
||||
@@ -156,7 +156,7 @@ export const LeftSidebar = ({onOpenProductSearch} = {}) => {
|
||||
className="bg-slate-50 hover:bg-slate-100 p-2.5 rounded-lg border border-slate-200 flex items-center justify-between text-xs transition-colors group"
|
||||
>
|
||||
<div className="flex-1 min-w-0 pr-2">
|
||||
<p className="font-bold text-slate-800 truncate">{filter.name}</p>
|
||||
<button className="font-bold text-blue-800 underline text-left" onClick={()=>applySavedFilter(filter)} aria-label={`${filter.name} filtresini uygula`}>{filter.name}</button>
|
||||
<p className="text-[10px] text-slate-500 truncate mt-0.5">
|
||||
👤 {filter.user} {filter.sehir && `• 📍 ${filter.sehir}`}
|
||||
</p>
|
||||
|
||||
@@ -4,7 +4,7 @@ import { INSTITUTION_TYPES_LIST, CITIES_LIST } from '../data/constants';
|
||||
import { Bot, ArrowLeft, CheckCircle2, HelpCircle, Building2, MapPin, Tag } from 'lucide-react';
|
||||
|
||||
export const SmartFilterBuilder = () => {
|
||||
const { savedFilters, addSavedFilter, deleteSavedFilter, setActiveTab, sektorListesi, savingCount, saveError } = useApp();
|
||||
const { savedFilters, applySavedFilter, addSavedFilter, deleteSavedFilter, setActiveTab, sektorListesi, savingCount, saveError } = useApp();
|
||||
|
||||
const [filterName, setFilterName] = useState('');
|
||||
const [selectedKurumTip, setSelectedKurumTip] = useState('');
|
||||
@@ -57,7 +57,7 @@ export const SmartFilterBuilder = () => {
|
||||
</div>
|
||||
<div>
|
||||
<h1 className="text-xl font-black text-slate-900">Kayıtlı Filtreler</h1>
|
||||
<p className="text-xs text-slate-500">Arama ölçütlerinizi hesabınıza kaydedin. E-posta bildirimleri Yönetim → Mobil PWA bölümünden ayrıca kurulur.</p>
|
||||
<p className="text-xs text-slate-500">Arama ölçütlerinizi kaydedin; soldaki filtre adına tıklayarak ihale listesine uygulayın. E-posta bildirimleri Yönetim → Mobil PWA bölümünden ayrıca kurulur.</p>
|
||||
</div>
|
||||
</div>
|
||||
<button
|
||||
@@ -96,7 +96,7 @@ export const SmartFilterBuilder = () => {
|
||||
{savedFilters.map(f => (
|
||||
<div key={f.id} className="p-3 bg-slate-50 border border-slate-200 rounded-lg text-xs hover:bg-slate-100 transition-colors">
|
||||
<div className="flex items-center justify-between font-bold text-slate-900">
|
||||
<span>🤖 {f.name}</span>
|
||||
<button type="button" className="text-blue-800 underline" onClick={()=>applySavedFilter(f)} aria-label={`${f.name} filtresini uygula`}>{f.name}</button>
|
||||
<button
|
||||
disabled={savingCount > 0} aria-label={`${f.name} filtresini sil`} onClick={async()=>{setSuccessMessage('');await deleteSavedFilter(f.id);}}
|
||||
className="text-slate-400 hover:text-red-600 text-xs"
|
||||
|
||||
@@ -1,3 +1,4 @@
|
||||
import {matchesRegion,matchesProcurement} from '../services/savedFilterCriteria';
|
||||
import { matchesTenderSearch } from '../services/tenderSearch';
|
||||
import React, { useState, useMemo, useEffect, useRef, useCallback } from 'react';
|
||||
import { useApp } from '../context/AppContext';
|
||||
@@ -18,7 +19,7 @@ export const TenderList = () => {
|
||||
searchQuery,
|
||||
kalemEslesen, itemSearchState, retryItemSearch,
|
||||
uyumEsigi,
|
||||
selectedCity,
|
||||
selectedCity,selectedRegion,setSelectedRegion,
|
||||
selectedAlimTuru,
|
||||
selectedSektorler,
|
||||
selectedCpvler,
|
||||
@@ -56,7 +57,8 @@ export const TenderList = () => {
|
||||
|
||||
// Dropdown Filters
|
||||
if (selectedCity && normTr(tender.sehir) !== normTr(selectedCity)) return false;
|
||||
if (selectedAlimTuru && tender.alimTuru !== selectedAlimTuru) return false;
|
||||
if (!matchesProcurement(tender,selectedAlimTuru)) return false;
|
||||
if (!matchesRegion(tender,selectedRegion)) return false;
|
||||
if (selectedSektorler.length && !selectedSektorler.includes(tender.sektor)) return false;
|
||||
if (selectedCpvler.length && !(tender.cpvBolum || []).some(b => selectedCpvler.includes(b))) return false;
|
||||
if (selectedKurumTipi && tender.kurumTipi !== selectedKurumTipi) return false;
|
||||
@@ -69,7 +71,7 @@ export const TenderList = () => {
|
||||
if (sortBy === 'distant') return compareTenderDeadlines(a,b,-1);
|
||||
return 0;
|
||||
});
|
||||
}, [tenders, searchQuery, kalemEslesen, uyumEsigi, selectedCity, selectedAlimTuru, selectedSektorler, selectedCpvler, selectedKurumTipi, quickFilter, sortBy, onlyValidOnly, now, favoriteKurumlar, oncekiGiris]);
|
||||
}, [tenders, searchQuery, kalemEslesen, uyumEsigi, selectedCity, selectedRegion, selectedAlimTuru, selectedSektorler, selectedCpvler, selectedKurumTipi, quickFilter, sortBy, onlyValidOnly, now, favoriteKurumlar, oncekiGiris]);
|
||||
|
||||
const gorunenTenders = filteredTenders.slice(0, gosterilen);
|
||||
const dahaVar = gosterilen < filteredTenders.length;
|
||||
@@ -78,7 +80,7 @@ export const TenderList = () => {
|
||||
// dizisine bağlansaydı havuz her yenilendiğinde (10 dk) kullanıcı listenin
|
||||
// başına fırlardı.
|
||||
useEffect(() => { setGosterilen(SAYFA_ADIM); }, [
|
||||
searchQuery, selectedCity, selectedAlimTuru, selectedSektorler,
|
||||
searchQuery, selectedCity, selectedRegion, selectedAlimTuru, selectedSektorler,
|
||||
selectedCpvler, selectedKurumTipi, quickFilter, sortBy, onlyValidOnly,
|
||||
]);
|
||||
|
||||
@@ -128,6 +130,7 @@ export const TenderList = () => {
|
||||
return (
|
||||
<div className="w-full min-w-0 flex-1 space-y-4">
|
||||
|
||||
{selectedRegion&&<div className="flex gap-3 items-center flex-wrap text-sm bg-blue-50 border rounded p-3"><span>İl bazında bölge filtresi: {selectedRegion}</span><button className="underline text-blue-800" onClick={()=>setSelectedRegion('')}>Bölge filtresini kaldır</button></div>}
|
||||
{/* Header bar above list */}
|
||||
<div className="bg-white rounded-xl border border-slate-200 p-4 shadow-xs flex flex-col sm:flex-row items-center justify-between gap-3">
|
||||
<div className="flex items-center gap-2">
|
||||
|
||||
@@ -1,3 +1,4 @@
|
||||
import {savedFilterCriteria} from '../services/savedFilterCriteria';
|
||||
import {mergeServerNotifications,acknowledgeNotifications} from '../services/notifications';
|
||||
import { searchTender } from '../services/searchTender';
|
||||
import { normalizeTenderSearch, matchingItemResults } from '../services/tenderSearch';
|
||||
@@ -80,6 +81,7 @@ export const AppProvider = ({ children }) => {
|
||||
return () => { iptal = true; controller.abort(); clearTimeout(zaman); };
|
||||
}, [searchQuery,kalemRetry]);
|
||||
const [selectedCity, setSelectedCity] = useState('');
|
||||
const [selectedRegion,setSelectedRegion]=useState('');
|
||||
const [selectedAlimTuru, setSelectedAlimTuru] = useState('');
|
||||
const [selectedKurumTipi, setSelectedKurumTipi] = useState('');
|
||||
// Çoklu sektör seçimi. Tek seçimlik eski API (selectedSektor) uyumluluk
|
||||
@@ -689,7 +691,19 @@ export const AppProvider = ({ children }) => {
|
||||
const addSavedFilter = newFilter => persistSavedFilters([{...newFilter,id:'sf-'+crypto.randomUUID(),user:userProfile.username},...savedFilters]);
|
||||
const deleteSavedFilter = id => persistSavedFilters(savedFilters.filter(filter=>filter.id!==id));
|
||||
|
||||
const applySavedFilter = filter => {
|
||||
try {
|
||||
const criteria=savedFilterCriteria(filter);
|
||||
setSearchQuery('');setSelectedCpvler([]);setQuickFilter('all');
|
||||
setSelectedCity(criteria.city);setSelectedRegion(criteria.region);
|
||||
setSelectedAlimTuru(criteria.procedure);setSelectedKurumTipi(criteria.institution);
|
||||
setSelectedSektorler(criteria.sector?[criteria.sector]:[]);setActiveTab('list');setSaveError(null);
|
||||
return true;
|
||||
}catch(error){setSaveError(error.message);return false;}
|
||||
};
|
||||
|
||||
const clearAllFilters = () => {
|
||||
setSelectedRegion('');
|
||||
setSearchQuery('');
|
||||
setSelectedCity('');
|
||||
setSelectedAlimTuru('');
|
||||
@@ -723,7 +737,7 @@ export const AppProvider = ({ children }) => {
|
||||
searchQuery,
|
||||
kalemEslesen, itemSearchState, retryItemSearch,
|
||||
setSearchQuery,
|
||||
selectedCity,
|
||||
selectedCity, selectedRegion,setSelectedRegion,applySavedFilter,
|
||||
setSelectedCity,
|
||||
selectedAlimTuru,
|
||||
setSelectedAlimTuru,
|
||||
|
||||
@@ -0,0 +1,19 @@
|
||||
const norm=value=>String(value || '').normalize('NFC').trim().toLocaleUpperCase('tr-TR');
|
||||
export const REGION_CITIES={
|
||||
'MARMARA BÖLGESİ':'Balıkesir,Bilecik,Bursa,Çanakkale,Edirne,İstanbul,Kırklareli,Kocaeli,Sakarya,Tekirdağ,Yalova',
|
||||
'EGE BÖLGESİ':'Afyonkarahisar,Aydın,Denizli,İzmir,Kütahya,Manisa,Muğla,Uşak',
|
||||
'AKDENİZ BÖLGESİ':'Adana,Antalya,Burdur,Hatay,Isparta,Kahramanmaraş,Mersin,Osmaniye',
|
||||
'İÇ ANADOLU BÖLGESİ':'Aksaray,Ankara,Çankırı,Eskişehir,Karaman,Kayseri,Kırıkkale,Kırşehir,Konya,Nevşehir,Niğde,Sivas,Yozgat',
|
||||
'KARADENİZ BÖLGESİ':'Amasya,Artvin,Bartın,Bayburt,Bolu,Çorum,Düzce,Giresun,Gümüşhane,Karabük,Kastamonu,Ordu,Rize,Samsun,Sinop,Tokat,Trabzon,Zonguldak',
|
||||
'DOĞU ANADOLU BÖLGESİ':'Ağrı,Ardahan,Bingöl,Bitlis,Elazığ,Erzincan,Erzurum,Hakkari,Iğdır,Kars,Malatya,Muş,Tunceli,Van',
|
||||
'GÜNEYDOĞU ANADOLU BÖLGESİ':'Adıyaman,Batman,Diyarbakır,Gaziantep,Kilis,Mardin,Siirt,Şanlıurfa,Şırnak',
|
||||
};
|
||||
const cityRegions=new Map(Object.entries(REGION_CITIES).flatMap(([region,cities])=>cities.split(',').map(city=>[norm(city),region])));
|
||||
export function matchesRegion(tender,region){return !region || cityRegions.get(norm(tender.sehir))===norm(region);}
|
||||
export function matchesProcurement(tender,procedure){return !procedure || [tender.usul,tender.ihaleUsulu,tender.alimTuru].some(value=>norm(value)===norm(procedure));}
|
||||
export function savedFilterCriteria(filter){
|
||||
const read=key=>{if(filter[key]!=null&&typeof filter[key]!=='string')throw new Error('Kayıtlı filtre ölçütü geçersiz.');return (filter[key] || '').trim();};
|
||||
const region=read('bolge');
|
||||
if(region&&!Object.keys(REGION_CITIES).includes(norm(region)))throw new Error('Kayıtlı bölge tanınmadı; filtreyi yeniden düzenleyin.');
|
||||
return {city:read('sehir'),region:region?norm(region):'',sector:read('sektor'),institution:read('kurumTipi'),procedure:read('usul')};
|
||||
}
|
||||
@@ -0,0 +1,21 @@
|
||||
import test from 'node:test';
|
||||
import assert from 'node:assert/strict';
|
||||
import {REGION_CITIES,matchesRegion,matchesProcurement,savedFilterCriteria} from '../src/services/savedFilterCriteria.js';
|
||||
import {CITIES_LIST} from '../src/data/constants.js';
|
||||
test('province-based regions cover all 81 cities exactly once and normalize Turkish names',()=>{
|
||||
const cities=Object.values(REGION_CITIES).flatMap(value=>value.split(','));
|
||||
assert.equal(cities.length,81);assert.deepEqual([...cities].sort(),[...CITIES_LIST].sort());assert.equal(new Set(cities).size,81);
|
||||
assert.equal(matchesRegion({sehir:' giresun '},'KARADENİZ BÖLGESİ'),true);
|
||||
assert.equal(matchesRegion({sehir:'Kayseri'},'KARADENİZ BÖLGESİ'),false);
|
||||
assert.equal(matchesRegion({sehir:''},'KARADENİZ BÖLGESİ'),false);
|
||||
});
|
||||
test('saved criteria preserves all five fields and rejects unrecognized regions',()=>{
|
||||
assert.deepEqual(savedFilterCriteria({sehir:' Giresun ',bolge:'karadeniz bölgesi',sektor:'Laboratuvar',kurumTipi:'Hastaneler',usul:'Açık İhale'}),{city:'Giresun',region:'KARADENİZ BÖLGESİ',sector:'Laboratuvar',institution:'Hastaneler',procedure:'Açık İhale'});
|
||||
assert.throws(()=>savedFilterCriteria({bolge:'Bilinmeyen'}),/tanınmadı/);
|
||||
assert.throws(()=>savedFilterCriteria({sehir:[]}),/geçersiz/);
|
||||
});
|
||||
test('procedure filter uses explicit source procedure, not inferred purchase category',()=>{
|
||||
assert.equal(matchesProcurement({alimTuru:'Mal Alımı',ihaleUsulu:'Açık İhale'},'Açık İhale'),true);
|
||||
assert.equal(matchesProcurement({alimTuru:'Mal Alımı'},'Açık İhale'),false);
|
||||
assert.equal(matchesProcurement({alimTuru:'Doğrudan Temin'},'Doğrudan Temin'),true);
|
||||
});
|
||||
Reference in New Issue
Block a user