Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[WIP] Update ActiveFedora to 12.0 #2256

Closed
wants to merge 2 commits into from
Closed

[WIP] Update ActiveFedora to 12.0 #2256

wants to merge 2 commits into from

Conversation

jcoyne
Copy link
Member

@jcoyne jcoyne commented Nov 22, 2017 •
edited
Loading

Waiting on releases of:

@mjgiarlo
Copy link
Member

bundler went 💥

@jcoyne jcoyne mentioned this pull request Nov 22, 2017
Closed
@jcoyne
Copy link
Member Author

jcoyne commented Nov 23, 2017

Related to samvera/hydra-head#425

@jcoyne jcoyne mentioned this pull request Dec 4, 2017
Closed
@jcoyne jcoyne force-pushed the update_af branch 2 times, most recently from dcc5d24 to a226be0 Compare December 8, 2017 04:51
@jcoyne jcoyne changed the title Update ActiveFedora to 12.0 [WIP] Update ActiveFedora to 12.0 Dec 8, 2017
@jcoyne
Copy link
Member Author

jcoyne commented Dec 8, 2017

Blocked by:
samvera/noid-rails#75
samvera/hydra-pcdm#246

@jcoyne jcoyne force-pushed the update_af branch 6 times, most recently from f3fc75e to 71b1c2c Compare December 8, 2017 15:18
@jenlindner
Copy link
Contributor

These were merged:
samvera/noid-rails#75
samvera/hydra-pcdm#246

@jcoyne jcoyne force-pushed the update_af branch 5 times, most recently from 9bfe9f0 to 39d8f80 Compare January 17, 2018 19:04
@jcoyne
Update ActiveFedora to 12.0
4f2ab1f 
And Hydra-head to 11.0
@cjcolvar
Copy link
Member

Blocked waiting for final release of Hydra-head 11.

@cjcolvar cjcolvar mentioned this pull request May 5, 2018
Closed
@afred
Copy link
Contributor

afred commented May 16, 2018 •
edited
Loading

During PR review session after tech call today, we labeled this as "backward incompatible" (a new label we just created) because it removes solrizer as a dependency, but Hyrax apps may be using Solrizer in indexers or anywhere elsewhere.

To be backward compatible, we probably need to ensure that solrizer is still a dependency, and we may want to wait until Solrizer methods issue proper deprecation warnings as well (samvera-deprecated/solrizer#45),

@stale
Copy link

stale bot commented Oct 11, 2018

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Oct 11, 2018
@no-reply
Copy link
Member

I have plans to get a clean AF 12.0 upgrade, so I'm closing this ahead of stalebot.

@no-reply no-reply closed this Oct 13, 2018
@no-reply no-reply deleted the update_af branch October 13, 2018 01:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
backward incompatible stale
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
6 participants
@jcoyne @mjgiarlo @jenlindner @cjcolvar @afred @no-reply