feat: dvt-alerts #65
feat: dvt-alerts #65
Conversation
| })); | ||
|
|
||
| setApiData(editedData); | ||
| } catch (error) {} |
There was a problem hiding this comment.
catch boş olmaması lazım. console.error verebilirsin ya da alert varsa mesala,
There was a problem hiding this comment.
bu sıfır fetch koyarsak ama her şey kullanırsak saçma olur, biz yeni hooks useFetch gerekiyor yarın beraber bakalım. Bu fetch kalsın
|
figmadaki eksik Create a New Alert button kaldı. koymamız gerekiyor. module.tsx açabilirsin tasarlama yapabilirsin bi de eğer data boş olursa DvtIconDataLabel koyabilir. (Alert button basarsa data baştan yapabilir) ayrıca sidebar 4 tane dropdown seçip sayfa alerts data çalıştırması lazım superset gibi aynısı olmalı. sidebar 4 tane dropdown seçilecek redux dispatch için gerekiyor dvt-app örneğimi olsun yeni dvt-sidebar oluşturabilirsin. redux dvt-sidebar object içinde 4 tane objectname yazalım dönüşü dvt-alerts olacak. sidebar redux yapacaksan yeni branch yapmana gerek yok, branch'teysen sayfa ve redux kodlayabilir |
…into feat/dvt-alerts
SUMMARY
AlertReportList replaced with DvtAlerts, DvtAlerts Page Created
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
TESTING INSTRUCTIONS
ADDITIONAL INFORMATION