benzend / goalboard-collab

0 stars 1 forks source link

Simplify auth logic, reduce overall complexity, follow syntax patterns #37

Closed benzend closed 5 months ago

benzend commented 5 months ago

Cleaned up a bunch of stuff and simplified lots of the logic:

  1. Converted the jwt confirmation into a method that returns a user object (or err) called Authorize
  2. Cleaned up our migration files - needed to revert to proper syntax and naming. Also added in the down.sql for the goal table
  3. Got login, register, and goals (GET) up and running
benzend commented 5 months ago

@GwartneyDev curious if we should merge this PR first since there's wayyyyy more changes.

I feel like we could easily port over your updated logic into the right functions with less friction

GwartneyDev commented 5 months ago

@benzend Yea that is exactly what I was thinking. Would be faster as well.

GwartneyDev commented 5 months ago

@benzend Go ahead and merge it and I can work through it all.