auth icon indicating copy to clipboard operation
auth copied to clipboard

Select auth methods

Open SimonLab opened this issue 3 years ago β€’ 7 comments

ref: #183

SimonLab avatar Mar 09 '22 10:03 SimonLab

Running mix ecto.reset and trying the run the application returns a constraint error: image

Fixed by deleting ald session/cookies from the browser

SimonLab avatar Mar 09 '22 11:03 SimonLab

Codecov Report

Merging #185 (1fca726) into main (ec88a03) will not change coverage. The diff coverage is n/a.

:exclamation: Current head 1fca726 differs from pull request most recent head b71e141. Consider uploading reports for the commit b71e141 to get more accurate results

@@            Coverage Diff            @@
##              main      #185   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           27        27           
  Lines          586       586           
=========================================
  Hits           586       586           

Continue to review full report at Codecov.

Legend - Click here to learn more Ξ” = absolute <relative> (impact), ΓΈ = not affected, ? = missing data Powered by Codecov. Last update ec88a03...b71e141. Read the comment docs.

codecov[bot] avatar Mar 09 '22 13:03 codecov[bot]

error while running mix ecto.reset: image

It looks like the error is link when using Fields to insert the person email.

solution:

  • The application compiled however I forgot to source my environment variables
  • So Fields didn't have the right application configuration
  • delete deps rm -r deps/
  • run again mix ecto.reset which recompile the project and reinitialise the database
  • delete existing cookie on your browser and the application run correctly again

SimonLab avatar Mar 16 '22 16:03 SimonLab

Interesting ... LMK if you need help debugging. πŸ”

nelsonic avatar Mar 16 '22 17:03 nelsonic

@nelsonic thanks, I just found out what was the issue, we just need to make sure the environment variables are sourced before the project is compiled! I've updated my previous comment with the solution

SimonLab avatar Mar 16 '22 17:03 SimonLab

Hi @SimonLab, πŸ‘‹ Keen to get at least the requirement captured by this PR. πŸ“ If we end up re-building auth using mix phx.gen.auth #207 ... πŸ€” we want to have as many of the features/requirements captured as possible up-front. πŸ’­ Please dedicate some time to this when you can. 🀞

P.S: obvs if you can get it working with tests, even better! 😜

nelsonic avatar Jun 17 '22 07:06 nelsonic

Let's re-visit this as part of the re-build #207 πŸ’­

nelsonic avatar Oct 14 '22 15:10 nelsonic