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

Issue 113 help center page #121

Merged
merged 32 commits into from
Jul 17, 2024
Merged

Issue 113 help center page #121

merged 32 commits into from
Jul 17, 2024

Conversation

MitanM
Copy link
Collaborator

@MitanM MitanM commented Jul 13, 2024

figma

Change Summary

This pull request follows up on issues #75 and #81 to implement the Help Center page. The Help Center page provides navigation to the Report a Problem page (#90) and the Contact page. The page reuses the tag component created in issue #81 and will include a header component similar to the one used on the Report a Problem page. This header component will be integrated into the report a problem page.

Changes Made
Help Center Page:
Created a new Help Center page within the pages/accout/help.
Added navigation tags for "Help Center" and "Contact" pages using the tag component from issue #81.
Header Component:
Created the header component from the Report a Problem page (#90).
Navigation:
Implemented navigation links within the Help Center page to direct users to the Report a Problem page and the Contact page.

  • Has two tag components for help center and contact pages.
  • Create or reuses a header component as seen in Report A Problem Page #90.
  • Uses a new page in the account subfolder/directory.

keyman015 and others added 29 commits July 9, 2024 08:40
- Remove unused class in the submit button as the disable prop is used
  instead of a class name to disable the button.
- Remove the writein interface to then import it.
- Fix data change effect causes infinite errors in console to be logged.
- Add back button in header that goes back to the previous page on the
  site using a router.
- Modify the submit button to use correct variant styling instead of
  custom styling.
- Adjust flex-box styling.
- Add new prop to write in component to pass classnames to the main div
  of the write-in component.
- Flex the report page to be more dynamic.
- Allow component to grow to the height to take up the remaining space.
- Add flex and flex-grow to fields in the write-in component.
- Add new custom upload image icon.
- Refactor upload image section to be its own component with the ability
  to hide image previews.
- Prevent report description from going below a certain height when
  uploading many images.
- Set default upper limit to inifinity (no limit) on image upload
  component.
- Set report page upper limit on image uploads to 5.
- Add option to remove the back button in the header through a prop.
- Add the ability to pass in the title as a prop to the compojnent.
- Remove import as image component was refactored.
- Add new icon for delete button.
- Add fade in effect with transparent background to distinguish the
  delete button.
- Fix bug preventing last uploaded image from being deleted as the
  effect prevented preview updates if the length was zero (no more
  images).
- Duplicate header component present in issue-90 branch which is more
  up to date so this current version is being deleted.
- Update imports to fix errors.
- Set the profile tag for report to send to the report page.
- Modify the main function to match the desired style.
- Fix imports by removing unused components.
Copy link

vercel bot commented Jul 13, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
penni ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 14, 2024 9:15am

@keyman015 keyman015 merged commit 3d26425 into main Jul 17, 2024
7 checks passed
@keyman015 keyman015 deleted the issue-113-Help_center_page branch July 17, 2024 10:19
@keyman015 keyman015 mentioned this pull request Jul 17, 2024
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants