-
Notifications
You must be signed in to change notification settings - Fork 34
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
1.0.1 Release (bug fix) #15
Merged
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
…to avoid breaking changes on deployment.
…e sure layout works.
…igning it as a parameter in an object is not working.
Set up next layout
* Made some minor changes to structure of app. * Setting up atoms needed for header and footer. * Added TechIsHirng logo and corresponding story. * Created Icon atom with corresponding stories. * Forgot to add a story for default layout so quickly threw one together. * Created header and added logo. * Added socials to footer with links. * Adding header and footer stories. * Forgot to add GitHub icon. * Some clean up and adding a way to easily clean install node_modules. * Added single navlink and main nav components and corresponding stories. * Added main nav to header without styling. Will style now. * Did some quick clean up and added a general link atom and its corresponding story. * Styled the desktop navigation and started work on mobile navigation. * Added mobile menu. Added Chakra UI to implement mobile menu. Will leverage Chakra UI components in the future. * Added actual links to pages. Pages currently do not exist. * Added typography and their corresponding stories. * Updating link component for logo. * Putting key in maps for navList.
* Moved main html tag to layout. * Just quickly made other pages for the other links for now. * Added ability to view tweets from the TechIsHiring Twitter account.
* Forgot to add Twitter feed story. * Started to do initial seo stuff. * Added next-sitemap and test generated a sitemap.
* Building the base api file. * Building api statues file. * Realized there was a React warning about uppercase in props and decided to quickly fix it. * Tested an example api to make sure what I was doing was correct. Will need to update useApiStatus as it's current form doesn't allow for intellisense. * Successfully added useApi hook. * Created an example component to show how the api layer works and moved the example api and the component into their own example folder. * Added a folder for endpoints and created a file with links to where I got this implementation. * Renamed implemenation source file to references.txt. * Adding an example of data fetching with the API layer that is nextjs specific. * Restructuring api folder to be more explicit and clear of content.
* Created card component and it's corresponding story. * Updated Twitter feed to use card component. * Added about page but getting a hydration error. Going to try and fix that right now. * Fixed issue with hydration error. When using Chakra UI text, you need to use 'as' to make into into the appropriate html tag.
* changed .env variable to .env.local and added posthog id. Will add posthog id to server as well. * Successfully added posthog to app.
* Added eslint from next and formatted project. * Forgot that stories were formatted weird.
* Added a basic endpoint for revue. Haven't tested yet. * Tested issues api and did some clean up to make it more usable in TypeScript. * Added subscribers api (not tested yet) and beginning putting together newsletter page. * Working on newsletter page. Adding necessary atoms so I can begin to test api. * Set up email entry. Can now test api. * Created a subscribe to newsletter molecule and migrated the code from newsletter content there. * Did some quick markup clean up. * Pivoting to making a next api to send request to revue because of cors issue. Testing now. * Got the api to work. Just need to clean up the app based on what the request sends back. * Added different states for how the email response. * Created newsletter entries organism to show the latest newsletters. * Doing some clean up on project.
* Fixing a bug where even though the api returned an error, the newsletter will say that it successfully went through. * Made top nav stick to the top. * Did active link for nav. * Added example for building layouts * Redid README file. * Added better image for TechIsHiring logo. * Pulled out newsletterentry from newsletter organism and made it a molecule. * Updated README to add Design Document. * Changed link to use next/link component and fixed some warnings throughout app. * Changed a console.log to output a string instead of the error because the error had the API Key.
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.
Fixing a quick bug and pushing out a release.