sciencehistory / chf-sufia

sufia-based hydra app
Other
9 stars 4 forks source link

Flip collections relationship (by upgrading to cc2 or migrating to hyrax) #453

Closed hackartisan closed 7 years ago

hackartisan commented 7 years ago

required for #349

hackartisan commented 7 years ago

As mentioned on slack, Sufia 7.3 does not run on cc2.

We have 2 options: upgrade now to hyrax or release a version of Sufia that does run on cc. I forayed very briefly into each option and I don't think either will be particularly less time-consuming than the other.

Upgrade to hyrax requires rails 5; that's pretty much where I stopped. Note this option isn't fully available until late April since hyrax 1.0 is in rc.

Releasing Sufia requires some code changes in Sufia to account for the flipped collections relationship. I made one such change and started testing the app a little; another one popped up and that's where I stopped. Note this option also requires some amount of community coordination as it will require releases of both cc and sufia.

We need to decide if this feature is worth the time it will take to pursue one of these paths and, if so, which path we want to take. I'm setting this aside for now to avoid spending too much time on it in the meanwhile.

jrochkind commented 7 years ago

@hackmastera Ah, so sufia can't be upgraded to CC2 without changes, it will require changes to sufia code beyond just updating the dependency?

Ack, all of these options are terrible.

Another option is I can continue figuring out how to do the collection facet under current sufia/CC. I started down that path, trying to figure out how to use callbacks to get solr index for works updated when a work is added/removed from a collection. After spending basically a whole day on it and not making much headway, I left it. The hydra code is convoluted.

I think it's possible we'll just have to give up on a collection facet for soft launch, the hydra stack does not make it feasible -- until we update to hyrax.

hackartisan commented 7 years ago

wontfix for now