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

Feature (Issue #41): Add new Admin Project Details (<AdminEditProjectForm />) page #43

Merged
merged 26 commits into from
Dec 15, 2020

Conversation

CodeMeKathy
Copy link
Collaborator

@CodeMeKathy CodeMeKathy commented Nov 30, 2020

Description

This pull request adds a new Admin Project Details page to match the redesigned mockup similarly.

Related Issue

#41

Acceptance Criteria

  • The Admin Project Details layout to match the redesigned mockup (the screen an authenticated user accesses to view the project details with contact information) looks similar to the redesigned mockup:

Admin Project List filled form (1)

Type of Changes

Type
🐛 Bug fix
✨ New feature
🔨 Refactoring
💯 Add tests
🔗 Update dependencies
📜 Docs

Updates

Before

This is a new page and did not previously exist.

After

AdminProjectDetails ProjectHub

Testing Steps / QA Criteria

  • In the terminal, run git checkout feature-admin-project-details to checkout to the feature-admin-project-details branch.
  • From the terminal, cd to the client directory and run npm start to load the server and start the browser.
  • In the browser, visit the <AdminEditProjectForm/> component by first selecting the ADMIN link on the top left of the navigation bar on the home page, at http://localhost:3000.
  • Once on the Admin Panel page, at http://localhost:3000/admin, select any of the listed projects to view its details in the <AdminEditProjectForm/>.

- Add adminProjectDetails scss stylesheet
- Reorg import statements
- Create _adminProjectDetails.scss style sheet
- Fixed Warning: validateDOMNesting(...): <h5> cannot appear as a child of <tr>
- Fixed Warning: validateDOMNesting(...): <th> cannot appear as a child of <thead>
@CodeMeKathy CodeMeKathy marked this pull request as draft November 30, 2020 18:36
@CodeMeKathy CodeMeKathy marked this pull request as ready for review November 30, 2020 18:38
@CodeMeKathy CodeMeKathy marked this pull request as draft November 30, 2020 18:40
@CodeMeKathy CodeMeKathy marked this pull request as ready for review November 30, 2020 18:47
@Rafna
Copy link
Member

Rafna commented Dec 15, 2020

Admin project details matches the design. Looking good.

@Rafna Rafna merged commit 5761007 into master Dec 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants