Re: [Snowdrift-dev] Weekly dev news 2017-07-22

2017-07-24 Thread fr33domlover
Oops I forgot the link: https://notabug.org/fr33domlover/shakespeare-sass ___ Dev mailing list Dev@lists.snowdrift.coop https://lists.snowdrift.coop/mailman/listinfo/dev

[Snowdrift-dev] Shakespeare splice support for SASS

2017-07-24 Thread fr33domlover
Hello people, I made a patch for shakespeare-sass which makes it support splices like the other shakespeare languages. I'd like to explain why and how because it's not easy to understand from my still ugly-and-undocumented code. First, how do these splices work in shakespeare in general?

Re: [Snowdrift-dev] Weekly dev news 2017-07-22

2017-07-24 Thread fr33domlover
skell compile. That certainly has drawbacks, but I think it's > important to get Sass working if at all possible. Fr33domlover is also > working on other possibilities, starting with a patch to shakespeare-sass. > > Thanks go to Ikomi for guiding this particular herd of cats towa

[Snowdrift-dev] [PATCH] Use appStripe as stripe client in handlers

2017-07-10 Thread fr33domlover
From: fr33domlover <fr33domlo...@riseup.net> This is a new version of the patch, it uses runStripe in appStripe. It avoids the StripeClient mess but still has a type error due to StripeRunner being having the 'forall io' thing. The errors can be fixed using type annotations probably, but

Re: [Snowdrift-dev] [PATCH] Use appStripe as stripe client in handlers

2017-07-10 Thread fr33domlover
I tried replacing all the 'StripeClient' with the actual type. It gives exactly the same type error. So, no harm in having the type alias. pgpK943rc6S5J.pgp Description: OpenPGP digital signature ___ Dev mailing list Dev@lists.snowdrift.coop

Re: [Snowdrift-dev] [PATCH] Use appStripe as stripe client in handlers

2017-07-10 Thread fr33domlover
On Mon, 10 Jul 2017 18:44:44 -0700 Bryan Richter <br...@snowdrift.coop> wrote: > To whom it may concern: > > fr33domlover and I know this doesn't compile.. it was submitted for > discussion. > > Now, on with the discussing: > > On Tue, Jul 11, 2017 at 03:57:

[Snowdrift-dev] [PATCH] Use appStripe as stripe client in handlers

2017-07-10 Thread fr33domlover
From: fr33domlover <fr33domlo...@riseup.net> --- crowdmatch/crowdmatch.cabal| 3 ++- crowdmatch/src/Crowdmatch.hs | 32 +++- website/src/AppDataTypes.hs| 6 ++ website/src/Handler/PaymentInfo.hs | 11 ++- website/src/Handler/U

[Snowdrift-dev] [PATCH] Add a real Stripe runner to crowdmatch test module, using test key

2017-07-10 Thread fr33domlover
From: fr33domlover <fr33domlo...@riseup.net> The runner isn't being used, I just added it there so that we can start writing tests with it. May wish to move it to a TestImport module once we split crowdmatch tests into multiple modules. --- crowdmatch/crowdmatch.cabal | 1 + crowdmatc

[Snowdrift-dev] [PATCH] Override Stripe secret key in test settings

2017-07-10 Thread fr33domlover
From: fr33domlover <fr33domlo...@riseup.net> --- website/config/test-settings.yml | 9 + 1 file changed, 9 insertions(+) diff --git a/website/config/test-settings.yml b/website/config/test-settings.yml index b2d43ca..ded169e 100644 --- a/website/config/test-settings.yml +++ b/w

[Snowdrift-dev] [PATCH] crowdmatch: Use (SqlPersistT env) monad to allow combining queries

2017-07-08 Thread fr33domlover
From: fr33domlover <fr33domlo...@riseup.net> Before this commit, crowdmatch actions run database queries separately inside them like a black box. That means in practice many applications of `runDB`, one per query, so a transaction per query. This is an unnecessary waste, and doesn't gai

[Snowdrift-dev] [PATCH] Port Snowdrift tests from HSpec to Tasty

2017-07-08 Thread fr33domlover
From: fr33domlover <fr33domlo...@rel4tion.org> --- website/Snowdrift.cabal | 12 +++- website/test/AlertsSpec.hs| 6 +++--- website/test/AuthSiteSpec.hs | 8 website/test/DiscourseSpec.hs | 35 --- website/test/HandlerSpec.hs

[Snowdrift-dev] [PATCH] Implement Alerts using Yesod.Core.Handler's messages

2017-07-01 Thread fr33domlover
From: fr33domlover <fr33domlo...@rel4tion.org> Yesod supports multiple messages now, which makes this possible. I left the AlertSpec test, just adapted it to the changes, but I must say it's somewhat useless now because Alerts is just a tiny tiny thin wrapper now. Also note the c

[Snowdrift-dev] [PATCH] Port Snowdrift tests from HSpec to Tasty

2017-07-01 Thread fr33domlover
From: fr33domlover <fr33domlo...@rel4tion.org> --- website/Snowdrift.cabal | 11 ++- website/test/AlertsSpec.hs| 6 +++--- website/test/AuthSiteSpec.hs | 8 website/test/DiscourseSpec.hs | 35 --- website/test/HandlerSpec.hs

[Snowdrift-dev] Chance for lightweight learning through patching the IRC bot

2016-12-24 Thread fr33domlover
Hey people, As some of you probably know, I wrote an IRC bot in Haskell from scratch, called FunBot. There's currently a single known instance of it, under the nickname "fpbot", running in #snowdrift and several other channels in Freenode. It's primary functional purpose is to help the free

Re: [Snowdrift-dev] [PATCH] Discourse SSO support

2016-12-06 Thread fr33domlover
y and confuse users once the Discourse instance starts getting filled with people and messages (or is it alreday? I didn't check). -- fr33 > Thanks!! > > - Jason > > On Mon, Dec 5, 2016 at 4:11 PM, <fr33domlo...@riseup.net> wrote: > > > From: fr33domlover <fr33do

Re: [Snowdrift-dev] [PATCH] Discourse SSO support

2016-12-05 Thread fr33domlover
ll see where that discussion goes. > > On Mon, Dec 05, 2016 at 04:13:42AM +0200, fr33domlover wrote: > > From: fr33domlover <fr33domlo...@rel4tion.org> > > > > --- > > website/Snowdrift.cabal | 7 > > website/config/routes| 2 ++ >

[Snowdrift-dev] [PATCH] Discourse SSO support

2016-12-05 Thread fr33domlover
From: fr33domlover <fr33domlo...@rel4tion.org> --- website/Snowdrift.cabal | 8 website/config/routes| 2 + website/config/settings.yml | 3 ++ website/src/Application.hs | 1 + website/src/Discourse.hs

[Snowdrift-dev] [PATCH] Discourse SSO support

2016-12-04 Thread fr33domlover
From: fr33domlover <fr33domlo...@rel4tion.org> --- website/Snowdrift.cabal | 7 website/config/routes| 2 ++ website/config/settings.yml | 4 +++ website/src/Application.hs | 1 + website/src/Discourse.hs

[Snowdrift-dev] SASS

2016-06-12 Thread fr33domlover
Hello, There are Haskell packages for using SASS in Yesod: - - I don't know if that's currently relevant. Just wanted to share in case it is. --fr33 signature.asc Description: PGP signature

[Snowdrift-dev] [PATCH] Make Snowdrift project wiki search redirect to the gitit wiki

2016-05-17 Thread fr33domlover
From: fr33domlover <fr33domlo...@rel4tion.org> --- website/Snowdrift.cabal | 1 + website/config/routes | 1 + website/src/Handler.hs | 4 3 files changed, 6 insertions(+) diff --git a/website/Snowdrift.cabal b/website/Snowdrift.cabal index 5cf461c..0994000 100644 --- a/w

Re: [Snowdrift-dev] Code reboot going into master

2016-03-19 Thread fr33domlover
On Fri, 18 Mar 2016 15:01:12 -0700 Bryan Richter wrote: > On Thu, Mar 17, 2016 at 11:20:22AM -0700, Bryan Richter wrote: > > As I may (or may not?) have mentioned earlier on this list, I > > started paring down the codebase to just the features we need for > > MVP. That has

Re: [Snowdrift-dev] [PATCH] User nicknames as public user IDs: Change UserR route

2016-03-07 Thread fr33domlover
On Mon, 7 Mar 2016 16:39:11 -0800 Bryan Richter wrote: > Doing so, of course, massively conflicts with the changes you (fr33) > are making; changes that are hard to make precisely because there's so > much dead weight. This causes me — and presumably you — some tension.

[Snowdrift-dev] [PATCH 2/2] Change all /u routes to use UserHandle

2016-03-07 Thread fr33domlover
From: fr33domlover <fr33domlo...@rel4tion.org> --- config/routes| 68 - src/Handler/NewDesign.hs | 5 +- src/Handler/ResetPassphrase.hs | 3 +- src/Handler/User/Balance.hs | 28 ++-- src/Handle

[Snowdrift-dev] [PATCH 1/2] 'userDisplayName' now takes a User, not Entity User

2016-03-07 Thread fr33domlover
From: fr33domlover <fr33domlo...@rel4tion.org> Username display is going to use only the 'userNick' and 'userName' fields. The ID column and the login ident are internal details. Therefore it becomes easier to the widely used 'userDisplayName' function to take just a 'User' value. T

Re: [Dev] Concerns regarding the mechanism

2015-07-30 Thread fr33domlover
to catch them (Yesod in general seems to take this approach seriously). All the m0ney related stuff is probably at least somewhat critical, so perhaps more work with types could replace some tests. Just a random idea, I don't know how the code actually works right now :P --- fr33domlover http