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

feat: contact profile page #48

Closed
wants to merge 48 commits into from
Closed

feat: contact profile page #48

wants to merge 48 commits into from

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 JoeKarow added new-feature New features or options. in-progress Issue is currently being worked on by a developer. labels Jun 19, 2022
@JoeKarow JoeKarow self-assigned this Jun 19, 2022
@JoeKarow JoeKarow modified the milestones: Contacts Module, Database Jun 19, 2022
@JoeKarow JoeKarow changed the base branch from main to develop June 19, 2022 03:56
JoeKarow and others added 10 commits June 20, 2022 19:11
@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[bot]
github-actions bot previously approved these changes Jun 23, 2022
View reviewed changes
@ghost
Copy link

ghost commented Jun 23, 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 the automerge Enable Kodiak auto-merge label Jun 23, 2022
github-actions[bot]
github-actions bot previously approved these changes Jun 24, 2022
View reviewed changes
github-actions[bot]
github-actions bot previously approved these changes Jun 24, 2022
View reviewed changes
github-actions[bot]
github-actions bot previously approved these changes Jun 25, 2022
View reviewed changes
@JoeKarow JoeKarow force-pushed the JoeKarow/47-feat-Contact-Profile-Page branch from 61ff2a0 to f1fbadc Compare June 25, 2022 01:33
github-actions[bot]
github-actions bot previously approved these changes Jun 25, 2022
View reviewed changes
@JoeKarow JoeKarow closed this Jun 26, 2022
@JoeKarow JoeKarow deleted the JoeKarow/47-feat-Contact-Profile-Page branch June 26, 2022 17:57
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

@JoeKarow JoeKarow

Labels
automerge Enable Kodiak auto-merge in-progress Issue is currently being worked on by a developer. new-feature New features or options.
Projects
None yet
Milestone
Contacts Module
Development

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