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.

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

initialisation du css,dependence et back #1

Merged
merged 6 commits into from
Dec 14, 2023
Merged

Conversation

Sarhane64
Copy link
Collaborator

No description provided.

adelka69
adelka69 previously approved these changes Dec 13, 2023
Copy link
Collaborator

@adelka69 adelka69 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nikel !

Copy link
Contributor

@jujuck jujuck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ajout d'un fichier service de connexion avec axios

frontend/package.json Outdated Show resolved Hide resolved
frontend/src/main.jsx Show resolved Hide resolved
import axios from "axios";

const connexion = axios.create({
baseURL: "https://some-domain.com/api/",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Non, c'est import.meta.env.VITE_BACKEND_URL + /api et non pas same-domain.com

<link
href="https://fonts.googleapis.com/css2?family=Montserrat&display=swap"
rel="stylesheet"
/>
<title>Vite App</title>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mettez tout de suite votre titre et votre favicon

{
path: "./admin",
element: <AdminPage />,
children: [{}],
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ajouter une page dans les children Admin

backend/seed.js Outdated
database.query("insert into profile(bio,avatar,alt) values (?,?,?)", [
faker.lorem.words(10),
faker.internet.avatar(),
"profile avatar",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Elle doit être unique

Romeoseb-dev
Romeoseb-dev previously approved these changes Dec 14, 2023
element: <Home />,
},
{
path: "./admin",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Attention, Admin n'est pas un enfant de app, mais un élement à part entière

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

le path n'est pas bon

Romeoseb-dev
Romeoseb-dev previously approved these changes Dec 14, 2023
Romeoseb-dev
Romeoseb-dev previously approved these changes Dec 14, 2023
@jujuck jujuck merged commit fb64fd4 into dev Dec 14, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants