Skip to content
This repository has been archived by the owner on Feb 5, 2023. It is now read-only.

JoeKarow/47 initial steps #69

Merged
merged 56 commits into from
Jun 28, 2022
Merged

JoeKarow/47 initial steps #69

merged 56 commits into from
Jun 28, 2022

Conversation

JoeKarow
Copy link
Owner

Pull Request type

Please check the type of change your PR introduces:

  • Bugfix
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no API changes)
  • Build-related changes
  • Documentation content changes
  • Other (please describe):

What is the current behavior?

Issue Number: #47

What is the new behavior?

Does this introduce a breaking change?

  • Yes
  • No

Other information

JoeKarow and others added 30 commits June 18, 2022 21:28
@JoeKarow
chore(global): 🔨 debug config
ef8695a
@JoeKarow
chore(web): 🔨 prisma seed instruction
08a6490
@JoeKarow
chore(web): 🔨 prisma seed script
62c8693
@JoeKarow
chore(web): 🔨 prisma schema building
e3b7169
@JoeKarow
feat(ui): ✨ getProviders hook for login button
3b96293
@JoeKarow
feat(ui): ✨ make UserButton dynamic based off session status
5f0ade0
@JoeKarow
refactor(ui): ♻️ use UserButton in main page header
df82bb4
@JoeKarow
chore(web): 🔨 install zod for validation
764b8e3
@JoeKarow
chore(web): 🔨 generate zod schemas
0e572ec
@JoeKarow
feat(web): ✨ start profile page & api call
eb24f7f
@JoeKarow
chore(web): 🔨 create fake db seed generator
9cd5eda
@JoeKarow
chore(web): 🔨 db schema updates
c94d189 
user table - add in ability to disable user from signin
@JoeKarow
refactor(web): ♻️ nextauth signin
297a333 
- adjusted scope for twitter login

- return user id & role properties to session on signin

- reject signin if user.disabled === true
@JoeKarow
refactor(web,lib): ♻️ move prisma client to netwerkr-lib package
23f6041 
This is to centralize the instance if PrismaClient to prevent multiple
instances from being spawned
@github-actions
Merge develop into JoeKarow/47-feat-Contact-Profile-Page
cf50142
@JoeKarow
refactor(web): ♻️ profile page from api -> getServerSideProps
c7ce4eb
@JoeKarow
chore(lib): 🔨 index file generation
4b71390
@JoeKarow
feat(lib): ✨ prisma data transformation
a513f4e 
prisma returns dates as objects, which Next does not like

This helper function converts given object keys to strings
@JoeKarow
refactor(ui): ♻️ removed some prop drilling for session params
9f90de0 
This should be okay - useSession hook shouldn't cause any additional renders
@JoeKarow
feat(lib): ✨ getter - basic contact info
dc9a657
@github-actions
Merge develop into JoeKarow/47-feat-Contact-Profile-Page
d176507
@github-actions
Merge develop into JoeKarow/47-feat-Contact-Profile-Page
f887403
@JoeKarow
chore(global): 🔨 stop yelling, eslint. You can find next/babel!
41f9a75
@JoeKarow
chore(global): 🔨 added: luxon; cross-installed other deps
21fd355
@JoeKarow
chore(lib): 🔨 schema updates - relation/optional fields
ed38f48
@JoeKarow
feat(ui): ✨ janky devtools - json viewer component
87c8f5d
@JoeKarow
refactor(lib): ♻️ contact getter & DateTime -> string converter
da4c1b3
@JoeKarow
feat(ui): ✨ start of contact profile display
ea731ee
@github-actions
Merge develop into JoeKarow/47-feat-Contact-Profile-Page
dcbd6c3
@github-actions
Merge develop into JoeKarow/47-feat-Contact-Profile-Page
df64604
@JoeKarow JoeKarow marked this pull request as draft June 26, 2022 18:00
github-actions[bot]
github-actions bot suggested changes Jun 26, 2022
View reviewed changes
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This pull request does not contain a valid label. Please add one of the following labels: ['breaking-change', 'bugfix', 'documentation', 'enhancement', 'refactor', 'performance', 'new-feature', 'maintenance', 'ci', 'dependencies']

@ghost
Copy link

ghost commented Jun 26, 2022 •
edited by ghost
Loading

👇 Click on the image for a new way to code review

  • Make big changes easier — review code in small groups of related files

  • Know where to start — see the whole change at a glance

  • Take a code tour — explore the change with an interactive tour

  • Make comments and review — all fully sync’ed with github

    Try it now!

Review these changes using an interactive CodeSee Map

Legend

CodeSee Map Legend

@JoeKarow JoeKarow added enhancement Enhancement of the code, not introducing new features. new-feature New features or options. in-progress Issue is currently being worked on by a developer. automerge Enable Kodiak auto-merge labels Jun 26, 2022
github-actions[bot]
github-actions bot previously approved these changes Jun 26, 2022
View reviewed changes
github-actions[bot]
github-actions bot previously approved these changes Jun 26, 2022
View reviewed changes
github-actions[bot]
github-actions bot previously approved these changes Jun 27, 2022
View reviewed changes
@JoeKarow JoeKarow marked this pull request as ready for review June 28, 2022 17:10
@JoeKarow JoeKarow merged commit 41759a5 into main Jun 28, 2022
@JoeKarow JoeKarow deleted the JoeKarow/47-initial-steps branch June 28, 2022 17:12
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@github-actions github-actions[bot] github-actions[bot] approved these changes

Assignees
No one assigned
Labels
automerge Enable Kodiak auto-merge enhancement Enhancement of the code, not introducing new features. in-progress Issue is currently being worked on by a developer. new-feature New features or options.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@JoeKarow