Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Feat/implemet create new alert #23

Merged
merged 19 commits into from
Jan 28, 2025
Merged

Feat/implemet create new alert #23

merged 19 commits into from
Jan 28, 2025

Conversation

ekumamatthew
Copy link
Contributor

@ekumamatthew ekumamatthew commented Jan 25, 2025 •
edited
Loading

Pull Request for SafeTrust - Close Issue #10

❗ Pull Request Information

This is a new feature/ create an alert screen

🌀 Summary of Changes

  • only created 3 files,

    1. custom input
    1. custom select
    1. create alert file
    1. route to create alert in the app folder

  • Add here the changes:

  • More changes:

🛠 Testing

Evidence Before Solution

Evidence After Solution

https://www.loom.com/share/3079290021e44670acb48492b027d664?sid=cc2bfd5c-c4b4-45f0-a953-2c7080ee3565

📂 Related Issue

This pull request will **close #10 ** upon merging.

Make sure to follow the Git Guidelines for Atomic Commits and read the Contributing Guide

The Pull request needs to have the format mentioned below in the Git Guideline

🎉 Thank you for reviewing this PR! 🎉

@ekumamatthew ekumamatthew marked this pull request as ready for review January 26, 2025 11:24
@JosueBrenes
Copy link
Contributor

Hi @ekumamatthew , there are several changes.

1 - You can leave the layout as it was please with the code that I will attach below

import type { Metadata } from "next";
import { Geist, Geist_Mono } from "next/font/google";
import "../../public/styles/globals.css";

const geistSans = Geist({
  variable: "--font-geist-sans",
  subsets: ["latin"],
});

const geistMono = Geist_Mono({
  variable: "--font-geist-mono",
  subsets: ["latin"],
});

export const metadata: Metadata = {
  title: "SafeTrust",
  description: "SafeTrust is a decentralized and secure platform P2P",
};

export default function RootLayout({
  children,
}: Readonly<{
  children: React.ReactNode;
}>) {
  return (
    <html lang="en">
      <body
        className={`${geistSans.variable} ${geistMono.variable} antialiased bg-white dark:bg-[#18181B]`}
      >
        {children}
      </body>
    </html>
  );
}

2 - Rename the component to CreateAlert (src\pages\SecurityManagement\createAlert.tsx)

3 - I would like the shape to look smaller like the image

image

4 - In "src\app\create-alert\page.tsx" paste this code below please

"use client";

import { AdminSidebar } from "@/layouts/Sidebar";
import { DashboardHeader } from "@/layouts/Header";
import { DashboardFooter } from "@/layouts/Footer";
import { SidebarProvider, SidebarInset } from "@/components/ui/sidebar";
import { ThemeProvider } from "@/components/theme-provider";
import CreateAlert from "@/pages/SecurityManagement/createAlert";

export default function CreateAlertPage() {
  return (
    <ThemeProvider attribute="class" defaultTheme="system" enableSystem>
      <SidebarProvider>
        <div className="flex min-h-screen flex-col">
          <div className="flex flex-1">
            <AdminSidebar />
            <SidebarInset className="flex flex-col w-full">
              <DashboardHeader />
              <main className="flex-1 p-8 pt-6 space-y-4">
                <CreateAlert />
              </main>
              <DashboardFooter />
            </SidebarInset>
          </div>
        </div>
      </SidebarProvider>
    </ThemeProvider>
  );
}

That's all for now, thank you very much

@JosueBrenes JosueBrenes merged commit e66b255 into safetrustcr:main Jan 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Implement Create New Alert On Notification Center
2 participants
@ekumamatthew @JosueBrenes