[PR] Initial implementation

[LuchoTurtle]

closes #2

It's still in-progress.

This PR adds a working example of showing the mutations related to a specific item.
The user double-clicks an item and the referring mutations are shown to the right.

• Add initial implementation
• Readme
• Add tests
• Add CI

[LuchoTurtle]

I still need to write the README.

[nelsonic]

Good work so far. But ...

Please scrap this and re-create the project from scratch and write the README.md while you are creating the App.
Retrospectively writing the docs is "no bueno" because you will always skip steps.
If the building was burning down and I could only save one thing it would be the README.md, not the /lib folder.
So please make the README.md as good as it possibly can be at every step of your journey.
Be concise but complete with the steps you write.
Don't bother with renaming the project from App to SpikePapertrail that will never be a requirement; ever.
Naming things is suuuper important. Repos are descriptive. Project files are namespaced "App".
Apologies if this has been inconsistent in some of our projects, e.g:
dwyl/phoenix-chat-example/blob/main/lib/chat_web/controllers/page_controller.ex
I haven't had time to retrospectively go back and "fix" everything.
But on the newer tutorials and our MVP, this has been App for a while ...
e.g: /dwyl/phoenix-todo-list-tutorial/blob/main/lib/app_web/controllers/item_controller.ex#L1
But even "Chat" as an App name isn't that bad - not much more typing than App ...
But the whole point of having App as the App's name
is so that when something works in one of the "experiments",
we can easily transfer the feature to our "main" App. 💭

[LuchoTurtle]

Nuked and started from scratch. It's still ongoing but I just want to give a status update.
Since I'm using the project without auth and api (since they're not yet merged and also makes it easier for this SPIKE), had to clone it from this version https://github.com/dwyl/phoenix-todo-list-tutorial/tree/ab4470d68f64ed8f5596dd09f3322193317b838e and had to add some tests/tinker with routes to get coverage to 100%. The project still 100% works.

[LuchoTurtle]

The initial implementation should be finalized.
Going to add CI to this PR.

[codecov]

Codecov Report

❗ No coverage uploaded for pull request base (main@b4c0315). Click here to learn what that means.
The diff coverage is n/a.

❗ Current head 6f40836 differs from pull request most recent head 62db7ef. Consider uploading reports for the commit 62db7ef to get more accurate results

@@           Coverage Diff            @@
##             main        #3   +/-   ##
========================================
  Coverage        ?   100.00%           
========================================
  Files           ?         9           
  Lines           ?        64           
  Branches        ?         0           
========================================
  Hits            ?        64           
  Misses          ?         0           
  Partials        ?         0           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[LuchoTurtle]

Just added CI and fixed warnings.
I haven't uploaded to Fly.io because I wager it makes sense only deploying after you review this PR.

[nelsonic]

@LuchoTurtle quite keen to get this shipped to Fly.io especially as all the code is already in place (Dockerfile, fly.toml etc.) Please assign to me when you feel it's ready for review. 🙏

[LuchoTurtle]

All yours 👍

[nelsonic]

@LuchoTurtle initial implementation looks good, thanks! 👍