22 Dec 2023 |
Gitlab Notifications | [snowdrift/elm-website] smichel17
deleted branch
add-license | 22:02:38 |
23 Dec 2023 |
Gitlab Notifications | [snowdrift/snowdrift] smichel17
pushed
1 commit
to
alpha-dashboard-tweaks:
| 23:12:51 |
Gitlab Notifications | [snowdrift/snowdrift] smichel17
commented on
merge request !227: Draft: Reorganize sections of dashboard
I'm not sure if this is working, but am just going to push it live, check there, and then make adjustments as needed in a follow-up PR
| 23:14:04 |
Gitlab Notifications | [snowdrift/snowdrift] smichel17
changed the title of
merge request !227 to Reorganize sections of dashboard | 23:14:08 |
Gitlab Notifications | [snowdrift/snowdrift] smichel17
pushed
1 commit
to
alpha-dashboard-tweaks:
-
c012189d
Fix "Not in scope: data constructor PayoutHistory" […]
| 23:56:12 |
24 Dec 2023 |
Gitlab Notifications | [snowdrift/snowdrift] smichel17
pushed
1 commit
to
alpha-dashboard-tweaks:
-
38ad6516
I think this will fix the type error […]
| 00:11:41 |
Gitlab Notifications | [snowdrift/snowdrift] smichel17
pushed
2 commits
to
alpha-dashboard-tweaks:
-
698751c8
Fix "Not in scope: data constructor PayoutHistory" […]
-
c4b50d54
I think this will fix the type error […]
| 00:15:14 |
Gitlab Notifications | [snowdrift/snowdrift] smichel17
pushed
1 commit
to
alpha-dashboard-tweaks:
| 00:21:19 |
Gitlab Notifications | [snowdrift/snowdrift] smichel17
commented on
merge request !227: Reorganize sections of dashboard
(after I finally fix the build, that is)
| 00:21:54 |
Gitlab Notifications | [snowdrift/snowdrift] smichel17
merged
merge request !227: Reorganize sections of dashboard | 00:52:13 |
Gitlab Notifications | [snowdrift/snowdrift] smichel17
pushed
9 commits
to
master:
-
62e87814
Guess at styling
-
698751c8
Fix "Not in scope: data constructor PayoutHistory" […]
-
c4b50d54
I think this will fix the type error […]
-
ecfe89fa
Finally fix build for real […]
-
de72a8df
Merge branch 'alpha-dashboard-tweaks' into 'master' […]
| 00:52:14 |
Gitlab Notifications | [snowdrift/snowdrift] smichel17
deleted branch
alpha-dashboard-tweaks | 00:52:14 |
Gitlab Notifications | [snowdrift/snowdrift] smichel17
commented on
issue #705: Dashboard improvements (ideally even for alpha)
| 01:36:52 |
Gitlab Notifications | [snowdrift/snowdrift] smichel17
commented on
merge request !227: Reorganize sections of dashboard
@chreekat I tried to pull out a helper function sum = foldr (+) 0 , but had trouble with the types.
Letting ghc infer the types, it can't infer the proper generic
• Couldn't match type ‘DonationUnits’ with ‘Cents’
Expected type: [DonationUnits]
Actual type: [Cents]
• In the first argument of ‘sum’, namely ‘(map getCharge payouts)’
In the expression: sum (map getCharge payouts)
In an equation for ‘totalCharged’:
totalCharged = sum (map getCharge payouts)
So I tried to type it as sum :: a => [a] -> a , but I'm not sure the right constraint to put on a , so I get this:
• Expected a type, but ‘a’ has kind ‘Constraint’
• In the type signature:
sum :: a => [a] -> a
In an equation for ‘getDashboardR’:
getDashboardR
= do { Entity uid _ <- requireAuth;
(patron, project) <- runDB
$ (,) <$> fetchPatron uid <*> fetchProject;
payouts <- runDB $ fetchPatronPayouts uid;
.... }
where
withMonthView (CrowdmatchDay d, amt) = (MonthView d, amt)
getDonation :: PayoutInstance -> DonationUnits
getDonation (_, _, donation, _) = donation
getFee :: PayoutInstance -> Cents
....
…so I decided not to bother.
| 12:30:06 |
Gitlab Notifications | [snowdrift/snowdrift] smichel17
commented on
merge request !227: Reorganize sections of dashboard
If I were more comfortable in the language, I'd probably define these inline as lambdas.
| 12:30:49 |
25 Dec 2023 |
Gitlab Notifications | [snowdrift/snowdrift] iko
commented on
issue #659: Migrate seafile to OSUOSL
Closing this issue, to reopen or create a new one when it is clearer this is needed or if Penpot will serve adequately. The issue description is also outdated — the Trello board has been retired for some time and this item is not being tracked in any open OSUOSL support ticket.
| 18:40:36 |
Gitlab Notifications | [snowdrift/elm-website] wolftune
commented on
merge request !9: Aaron's additions and edits to non-flo-specific crowdmatching
I made updated edits that are both here and at the shared doc https://docs.snowdrift.coop/6JCdJNobQteXjtAYMpDTqg?both
| 20:16:49 |
27 Dec 2023 |
Gitlab Notifications | [snowdrift/snowdrift] chreekat
commented on
merge request !227: Reorganize sections of dashboard
I recognize the first issue, although I can't remember what causes it: In some cases, GHC won't fully generalize the type of function defined in a where clause, and I think it might have something to do with one of the many language extensions that Yesod relies on.
The solution is to give it an explicit type signature, which is what you tried.
But a => [a] -> a says "Given some constraint named a , turn a list of a into a single a ". Things to the left of the fat arrow => must have kind Constraint, so it infers that before trying the rest of the type signature.
What would have worked is Num a => [a] -> a .
But what have worked even better would be to use the sum defined in the Prelude. https://hackage.haskell.org/package/base-4.19.0.0/docs/Prelude.html#v:sum :)
| 13:34:19 |
Gitlab Notifications | [snowdrift/snowdrift] chreekat
commented on
merge request !227: Reorganize sections of dashboard
This is more readable.
What would truly be better is to upgrade PayoutInstance to a real sum type, instead of a type synonym of a 4-tuple.
| 13:34:19 |
28 Dec 2023 |
Gitlab Notifications | [snowdrift/snowdrift] chreekat
pushed
3 commits
to
b/local-docker (new branch):
-
f0a0cd83
Don't rely on being in a git repo
-
6bb4b049
Add Dockerfile for local development
-
5cf13d33
Explain how to use the local build image
| 15:08:06 |
Gitlab Notifications | [snowdrift/snowdrift] chreekat
opened
merge request !229: Docker image for local development
See README.
| 15:09:20 |
Gitlab Notifications | [snowdrift/snowdrift] chreekat
pushed
1 commit
to
b/local-docker:
-
c388c012
Explain how to use the local build image
| 15:10:23 |
Gitlab Notifications | [snowdrift/snowdrift] chreekat
pushed
1 commit
to
b/local-docker:
-
7e601419
Explain how to use the local build image
| 15:11:44 |
Gitlab Notifications | [snowdrift/snowdrift] smichel17
commented on
merge request !229: Docker image for local development
It worked, but the workaround does not appear to do anything on my system; I get,
/bin/sh: line 0: ulimit: open files: cannot modify limit: Operation not permitted
That said, it also installed fast enough, so no issues here either.
| 18:06:22 |
Gitlab Notifications | [snowdrift/snowdrift] smichel17
commented on
merge request !229: Docker image for local development
I think I might have broken my docker install a while back when I was trying to run MusicBrainz locally and was setting up volumes at non-standard locations.. so I'm running into this error in step 15 (installing GHC), which I will need to spend some time tracking down.
Error processing tar file(exit status 1): write /home/smichel/.ghcup/ghc/8.0.2/lib/ghc-8.0.2/template-haskell-2.11.1.0/libHStemplate-haskell-2.11.1.0_p.a: no space left on device
I don't think this is an issue with the Dockerfile, but if you think it is (or have more ideas before I figure it out myself), let me know!
| 18:06:22 |
Gitlab Notifications | [snowdrift/snowdrift] unclechu
commented on
merge request !229: Docker image for local development
Not really directly related to your change but:
- In case the path to the script contains any spaces it can end up with being a wrong path or “too many arguments for cd” kind of error. But it won’t work as intended.
- If
dirname fails (e.g. mistyped command dirnmae ) projRoot will end up being your home directory (cd call with no arguments), the error will be masked. Depending on what the following script commands are a bunch of garbage artifacts can be created in your home dir as a side-effect.
If what you want is to set it to the project root relative to the script file this should be failproof:
SCRIPT_DIR=$(dirname -- "${BASH_SOURCE[0]}")
cd -- "$SCRIPT_DIR"
| 21:45:12 |
29 Dec 2023 |
Gitlab Notifications | [snowdrift/snowdrift] chreekat
commented on
merge request !229: Docker image for local development
Haha I have no idea what's happening here. Maybe a difference in the kernel.
| 06:31:49 |
Gitlab Notifications | [snowdrift/snowdrift] chreekat
replied to a thread on
merge request !229: Docker image for local development
Yeah, sounds like a you problem unfortunately.
| 06:32:26 |
Gitlab Notifications | [snowdrift/snowdrift] chreekat
commented on
merge request !229: Docker image for local development
Thanks! projRoot is used in a few different places in the script, so I probably just want
projRoot=$(dirname -- "${BASH_SOURCE[0]}")
| 06:37:25 |
Gitlab Notifications | [snowdrift/snowdrift] unclechu
commented on
merge request !229: Docker image for local development
Oh yeah, that’s actually what I meant. There was a cd inside the subshell interpolation, not outside of it.
| 06:41:56 |