Select auth methods

[SimonLab]

ref: #183

[SimonLab]

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

Fixed by deleting ald session/cookies from the browser

[codecov]

Codecov Report

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

❗ 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.

[SimonLab]

error while running mix ecto.reset:

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

[nelsonic]

Interesting ... LMK if you need help debugging. 🔍

[SimonLab]

@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

[nelsonic]

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]

Let's re-visit this as part of the re-build #207 💭