Fork me on GitHub
#onyx
<
2018-02-26
>
eelke10:02:06

Sure I'll see what I can do

michaeldrogalis14:02:30

@eelke If it's a huge pain, we can cut the release as is. Worth investigating, though.

eelke14:02:11

Yeah I need just need one build with the kill button removed. That is why I made it a compiler option

eelke14:02:44

I am not that familiar with om-next, so not sure how to best pass a run-time variable to clojurescript

eelke14:02:43

But I can relate to your argument

michaeldrogalis14:02:30

Okay, let's see what we come up with then.

eelke14:02:13

ok thanks 🙂

eelke15:02:28

As an alternative you can point me in the right direction and have a go. Would it be something like a get request from the client to the server for some app-state value?

michaeldrogalis16:02:18

I'm not sure to be honest - front-end isn't my area.

michaeldrogalis16:02:45

One thing we could to do to mitigate the pain it to factor out that build profile into a top level def in the project.clj

niamu16:02:51

@eelke yeah, the basic idea is that you’d want the server to hold the environment variable so that the client can query it for whether or not the kill button should be displayed/enabled. I can look at it by tomorrow as well.

michaeldrogalis16:02:46

Sounds good. If it ends up being a huge pain we can go with the original approach, but that idea sounds fine, too.

lucasbradstreet17:02:45

@niamu and ideally you would check it on the back end when they kill-job, to prevent client side hacks allowing you to kill jobs. It isn’t really written to protect against its users though.

niamu17:02:32

yeah, I’m sure I can do all of that by tomorrow. Assuming no one else beats me to it.

michaeldrogalis17:02:08

The dashboard really could use a wash at some point. Our front-end-fu was as poor when we wrote it as it is now, ha.