Edit page fetch - #78
Conversation
MelanyCristy
left a comment
MelanyCristy
left a comment
There was a problem hiding this comment.
I requested a few things for make the cleaner
| import ProjPanel from "./components/Projects/ProjPanel"; | ||
| import EditProject from "./components/Projects/EditProject"; | ||
|
|
||
|
|
There was a problem hiding this comment.
Blank space, please remove
|
|
||
|
|
||
| import Navbar from "./components/Navbar"; | ||
|
|
There was a problem hiding this comment.
Blank space, please remove
| const loggedInRouter = ( | ||
| <Router> | ||
|
|
There was a problem hiding this comment.
Blank space, please remove
| </Route> | ||
| </Switch> | ||
|
|
There was a problem hiding this comment.
Blank space, please remove
| <div class="container-fluid"> | ||
| <div className="form-group col-md-5"> | ||
|
|
||
|
|
There was a problem hiding this comment.
Blank space, please remove. It shouldn’t be spaces between divs only one at the end of each code element. Just for have in mind :D
| /> | ||
|
|
||
| </div> | ||
|
|
There was a problem hiding this comment.
Blank space, please remove
|
|
||
| </div> | ||
|
|
||
|
|
There was a problem hiding this comment.
Blank space, please remove
| import React from "react"; | ||
| import { Link } from "react-router-dom"; | ||
| import ProjectsApi from "./../../api/ProjectsApi"; | ||
| import "./MyProjects.css"; |
There was a problem hiding this comment.
Why removing this? Don't we need this css file? if don’t then i guess that file should be removed
|
|
||
|
|
||
| </p> | ||
| <div class="container"> |
There was a problem hiding this comment.
Change "class" to "className"
| <div className="form-group col-md-4"> | ||
| <label>Advanced Payment</label> | ||
|
|
||
|
|
There was a problem hiding this comment.
Blank space, please remove :)
pratima-shegokar
left a comment
pratima-shegokar
left a comment
There was a problem hiding this comment.
After clicking edit button
please solve the issue .
|
Cancelling the pull request because working functionality merging with pull request #81 . The pull request have the same file edits and fetching the values and merged with abed branch . |
|
Closing |
3 participants
Functionality to fetch details from the project panel page and edit them in the edit page.