TechIsHiring website release 2.0.7 #161
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Added GitHub Action that runs tests and linters on PRs going to dev and main. * Updating code quality GitHub Action to remove strategy section. * Forgot to remove strategy from linter GitHub Action. * Added specific GitHub Action for running Cypress tests. * Added proper name to Cypress GitHub Action. Updated Cypress to run tests on chrome. * Changing cy.visit to check if this would solve the failing test. * Trying to add timeout to fix cypress error. * Adding start to Cypress GitHub Action. * Adding GitHub Action to build next project before running Cypress. * Removing npm build to try and run dev server to run Cypress test.
* finished building the hero section, need to review * made suggested changes * made small changes on index.ts
* Added prettier and prettier-plugin-tailwindcss, added baseline rules, ran files through prettier * Moved eslint-disable-next-line rule down This is correcting an error caused by prettier formatting.
* feature: added the contact page and added the head component to particles * feature: completed the contact page and made it responsive * feature: added the contact page and added the head component to particles * fix: fixed issues generated in pr but left the nextjs config issues unresolved * fix: added stories to the storybook * fix: fixed issue generated in the pr * fix: fixed issue generated in the pr --------- Co-authored-by: Chad Rhonan Stewart <chadrhonanstewart@gmail.com>
* Updated eslintrc file to remove rules that are now being managed by prettier. * Forgot that I had deleted the noEmitOnError in tsconfig.
* feat: completed the header component * addressed merge conflicts * fix: making recommended changes * fix: adding back the accessibilit icon on main nav
…k alt text and updated PR Template (#71) * Refactored icon to no longer have a link component inside of it and updated Footer component accordingly. * Added external link alt text to useNav and updated mainNav to use external link alt text from hook. Need to work on type definitions to force defining an external link alt text when defining an external link. * Updated Pull Request Template to add a guide to linking issues so that they automatically close when the PR is merged. * Forgot to update story on icon. Removed the Link story.
* feature: added the contact page and added the head component to particles * feature: completed the contact page and made it responsive * feature: added the contact page and added the head component to particles * fix: fixed issues generated in pr but left the nextjs config issues unresolved * fix: added stories to the storybook * feature: added about page * feature: added the about page * fix: fixed issues generated in the PR * feat: added a preloader to the tweet feed * fix: fixed issues generated in the PR * fixed contact-stories issue * feat: added about page * fixed issues generated in PR * fixed issues generated in PR * refactored the about page * fixed issues generated in the PR * fix: fixed issues generated in the PR --------- Co-authored-by: Chad Rhonan Stewart <chadrhonanstewart@gmail.com>
* increase the height of the image * . * Delete yarn.lock Just deleting the yarn.lock file. --------- Co-authored-by: Chad Rhonan Stewart <chadrhonanstewart@gmail.com>
* feat: completed the header component * addressed merge conflicts * fix: making recommended changes * fix: adding back the accessibilit icon on main nav * feat: adding why-choose section of the home page * fixed the large hero section * fix: added changes to why choose us section
* feat: completed the header component * addressed merge conflicts * fix: making recommended changes * fix: adding back the accessibilit icon on main nav * updating my dev branch * feat:adding footer section * fix: added requested changes to the component
…t link anywhere. (#83) * Fixed issue where social links where not working. * Fixed issue where Contact Us button in desktop navbar did not have a link. * Added external link icon to newsletter link in footer. * Forgot that 'Contact Us' text was hard coded and removed that to dynamically use the text from the use-nav hook.
* Initial commit to resolve issue #80. * Replaced hero background image and did some clean up to the app. * Fixing some typos because they irk me, lol. * Updated the README to add a link to the dev environment.
* Added useMainNav to footer so that when the nav hook is updated, both the main nav and the footer is updated automatically. * Commented out search bar until a decision is made on the Twitter API. Very likely we won't use it further but until a decision is concrete, let's keep the code. * Updated storybook titles to match previous storybook titles. * Updated structure of app so that why choose section is not embedded in another folder. Also created story for why choose section. * Fixed issue where contact page image was not fitting design properly in smaller windows sizes. * Fixed issue where underline would appear under contact us button. * Made some changes to about page where media query was at 768px instead of 1024px.
* Starting to make the hire-chad page. * Continuing work on hire-chad page. Added image and about to add content. * Added basic set up for hire-chad page. * Doing some updates. * Making my profile mobile friendly. * Just cleaning up some css. * Added some links to about page. * Deleted test image to clean up image folder. * Fixing my profile pic and the head content. * Fixed meta tag stuff. * Added link to hire chad page in about page. * Updated social links for hire chad page. * Updated hire chad page with description and some minor css updates. * Fixed issue with footer nav being too small. * Added stories for the hire chad page. * Fixing an issue with about page. * Fixed another issue I found with the about page.
* Beginning work to update api layer. * Updated API Layer to use fetch api instead of axios. * Updated api layer documentation and deleted newsletter page and related api files. * Adding blank file to external-apis folder to preserve the folder structure in git. * Removing newsletter issue api return types and putting an example in place for when new types are added. * Removing newsletter components and respective stories. * Forgot to remove axios from the README. * Added async & await to get example request documentation.
* Upgraded to Next.js 13 and TypeScript 5 * Starting conversion of link tag to fit Next.js 13 standard. * Updated interfaces, moving url to navlink interface. Continuing update to Next.js 13 standard. * Completed Link conversion to Next.js 13 standard. Removed newsletter entry component as I forgot to remove it in earlier PR. * Updating image in hero section based off of Next.js 13 warning. * Fixed issue with storybook failing to build because of a React Docgen error involving TypeScript 5. Current solution will be easy to remove once the issue is resolved or in upgrading to Storybook 7. * Updated storybook to latest stable version. * Moved active link from link interface to navlink interface and removed reference from atom. Fixed a few other minor issues.
* Did initial work to set up Contact page. Successfully integrated Sendgrid. Just need to set up sending the email through the form now. * Working on creating email endpoint. * Added the endpoint function to the function. Now need to verify data. * Successfully added endpoint to form. Just need to test that it sends the data correctly. * Successfully sent message to backend. Just putting together email now. * Successfully make contact us form functional. Will likely need to work on more validation. * Updating documentation for creating api endpoints. * Adding max length to input boxes so messages aren't too big. * Added error messages to each input field in contact us page. * Added form validation to contact us form. * Added messages on different api states, a success message and an error message. * Fixed issue where div would become small when error message appeared on mobile. * Fixed an issue that I caused when trying to fix the previous issue, lol. Also removed base url cause I just realized I don't need that, double lol. * Added Zod and SendGrid to list of technologies in README. Added guard clause to process-email serverless function to not send an email if the request is invalid. * Making some changes based on code review. * Forgot to clean something up, lol.
* Fixing an issue with contact us page. * Change I made messed up the error message window. Had to fix that really quickly.
…107) * Fixed issue where height in hero page was a static number. * Added spinner to contact us form. Updated Spinner for Twitter Feed. * Updated Spinner to be it's own atom and made storybook story for it.
* Added highlight content component for TechIsHiring highlights. * Added basic structure of TechIsHiring highlights. * Completed basic styling of highlights component. Just adding loading state and choosing tweets to highlight. * Completed loading state for tweets. * Added specific Tweets to be featured. * Fixed issue with hero section where it was too large in desktop view. * Added stories for highlights details and highlights content. Also made some minor updates to other stories titles. * Adjusting TechIsHiring Highlights header text * Updating stories that were .jsx to .tsx. * Updated headers for sections of front page so they are more similar to each other. * Updating social card location because it is no longer working. * Adding a social card for the hire-chad page.
* Fixed issue where the site would stretch to fill screen size and look distorted at large screen sizes. * Moved padding in header component so that the header still has proper padding beyond max screen size. * Updated cypress test because header changed and forgot cypress test was checking for it. * Changed max screen width to 1920px from 1440px. * Removing border from card component and updating comments in tailwind config.
…l the available space when the window size is less than 1920px. (#118)
* add cypress asset folders to .gitignore * update cypress to latest * set up Cypress component testing * remove red squiglies * add custom cy.validateLink command for this project * add data-cy attributes for testing * add validateFooter util and footer component test * test homepage footer from e2e homepage test * minor tweaks * use cy.within() to simplify selector code * add check-ts for easy typescript validation * add declarations.d.ts
Fixed Markdown formatting
… h1 (#151) * refactor: import Heading instead of Text from chakra-ui for rendering h1 * edit cypress e2e testing for heading vs text
…153) * configure baseUrl in one place for e2e tests * update Cypress version to latest * add component testing to the github action * add cypress project id * add steps to run tests in parallel * skip building * skip building * remove extra branch, update container count
* adding type validation with Zod for process-email * Feature: Type validation with zod for process-email api
* Fix Code of Conduct link * Revert "Fix Code of Conduct link" This reverts commit 7279538. * Fix for issue 159
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.