Uku Taht
Uku Taht
> I know you didn't change how the Dockerfile is build but still worth testing I think
Nice, let's do it!
Good clues, thank you!
Let's not merge this until the actual release is live. In the release candidate announcement, we can link people to this PR for instructions how to enable the permissions required...
What's the minimum docker version for this to work? Maybe we need to add that under `Requirements`
> I've always thought the techincal debt we have is not using the Phoenix Route helpers which automatically url-encode your params. So instead of: ```elixir |> redirect(to: "/#{URI.encode_www_form(site.domain)}/settings/danger-zone") ``` We...
> This pull request is now ready to be reviewed, @plausible/developers. Back-end support for intervals is already in production, and it's playing perfectly with the master version of the dashboard...
:+1: Overall My only concern would be that elixir standard library does not provide functionality to manipulate dates and times. I wonder how far these koans could go when the...
Thank you for the suggestions @DaneWeber! I agree with @iamvery here. Showing the atom is a good improvement to the exercise but I would keep the double-equals. In my experience...
Strongly in favour. For people starting out with Elixir this is probably the best way. Before merging, does CodeSpaces have a convenient way to launch it with a link? Would...