Skip to content
Snippets Groups Projects

feat: persist report on the backend

Merged Samed requested to merge feat/report_backend into main
1 unresolved thread

Insight are now able to be saved in the db. Frontend properly fetches and shows the insights

Merge request reports

Pipeline #142904 passed

Pipeline passed for d6878f4f on feat/report_backend

Approval is optional

Merged by Dennis CollarisDennis Collaris 4 months ago (Dec 9, 2024 1:35pm UTC)

Merge details

  • Changes merged into master with 4bad3086.
  • Deleted the source branch.

Pipeline #142916 passed

Pipeline passed for 4bad3086 on main

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
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.');
  • Samed added 1 commit

    added 1 commit

    • d6878f4f - chore: added correct alerts & confirmation msg

    Compare with previous version

  • Dennis Collaris approved this merge request

    approved this merge request

  • :tada: This MR is included in version 1.120.0 :tada:

    The release is available on GitLab release.

    Your semantic-release bot :package: :rocket:

  • Dennis Collaris mentioned in merge request !251 (closed)

    mentioned in merge request !251 (closed)

  • Please register or sign in to reply
    Loading