feat: persist report on the backend
1 unresolved thread
1 unresolved thread
Insight are now able to be saved in the db. Frontend properly fetches and shows the insights
Merge request reports
Activity
requested review from @decode
assigned to @Balcioglu
added 9 commits
-
dd2c4767 - 1 commit from branch
main
- c0b3a7d0 - feat: report backend implementation
- e0156142 - feat: updated reportingform
- 5541f407 - chore: updates
- 99b7c111 - feat: frontend receives from ws
- 3eafc3ce - chore: remove webpage alerts
- 9d73651b - feat: add import/export capabilities to variable nodes
- fac6a882 - chore: style fixes and missing code fixes
- 9881c8b5 - fix: insight saves properly in fe & be
Toggle commit list-
dd2c4767 - 1 commit from branch
6 category: MonitorType; 9 setAdding: (val: false | MonitorType) => void; 10 setActive: (val: string) => void; 11 type: InsightType; 7 12 }; 8 13 9 14 export function AddItem(props: Props) { 10 const [value, setValue] = useState<string>(''); 15 const [name, setName] = useState<string>(''); 16 const [description, setDescription] = useState<string>(''); 17 const dispatch = useAppDispatch(); 18 const session = useSessionCache(); 19 20 const handleSave = async () => { 21 if (!name.trim()) { 22 alert('Please enter a name.'); At a couple of spots we still use native javascript alerts, it would be better to use our own addError method. Leo already did some of those in commit 3eafc3ce, but I see a few alerts still remain. Could you update these as well?
changed this line in version 3 of the diff
added 1 commit
- d6878f4f - chore: added correct alerts & confirmation msg
This MR is included in version 1.120.0The release is available on GitLab release.
Your semantic-release bot
mentioned in merge request !251 (closed)
Please register or sign in to reply