Skip to end of metadata
Go to start of metadata

You are viewing an old version of this page. View the current version.

Compare with Current View Page History

« Previous Version 11 Next »



HomeDeveloper ResourcesCheck-in Notes 
 Table of Contents



We'll record discussions and updates here in case there's anyone who misses the call

Week 1


Updates Mar 7

Issue #5461 (Anna) - batch edit not working with valkyrie

  • just that batch edit doesn't work
  • got nurax-pg up and running

Issue #5462 (Eliot) - editing parent work error

  • got nurax-pg up and running
  • got error following steps, but it is different
  • working on figuring that one out 

Issue #5480 (Trey) - file upload multiple problems

  • issue was filesets don't have a thumbnail
  • there are multiple errors
  • FIXED: valkyrie ingest job was failing because test and app worked a little different.  Can't copy carrier wave stream directly into a file. 
  • FIXED: characterization wasn't working because not able to set file set id 
  • FIXED: missing title 
  • derivatives don't run
  • rubocop doesn't like long method

Issue #5449 (Lynette) - nested collection form exceptions

  • working on nested collection form issues
  • FIXED: method missing for find
  • working on #nestable? method missing

Issue #6490 (Shaun) - UI consistency

  • Created a PR for a changed label.
  • Would recommend taking out icons from action menu

Daniel

  • planning to work with dassie with valkyrie setup
  • will look at setting up nurax-pg

Chris

  • looking to pick up something tomorrow

Updates Mar 8

Anna (#5461) - batch editing

  • pairing with Eliot
  • batch edit will need to be converted to a change set
  • perhaps it would be better to tear it out
  • There's another project for batch editing - bulkrax. Maybe putting bulkrax into hyrax is a potential?
  • Bulkrax may be more for bulk ingest.
  • Julie may know if there's something on the roadmap to do something for batch edit
  • Batch edit can be turned off. Maybe we can just make it so if Valkyrie's turned on, batch edit is left flipped off? Toggle based off Hyrax.config.disable_wings.


Chris (#5463) - missing _pcdm_partials

  • got docker setup working
  • taking a look at searches that return pcdm collections
  • looks like it has something to do with routes
  • also may need to override a blacklight helper for path resolution
  • look at it more tomorrow
  • would it work to add partials instead of overriding by rendering the _collection partial from _pcdm_collection partial.  Probably would see similar issues.


Daniel (#4788) - fileset id not indexed

  • thumbnail ids are being set on the model, but not indexed
  • first step is to index the fileset id
  • next step is to get access to the files
  • worked on getting nurax-pg docker environment be easier


Eliot (5462) - error editing with parent work

  • working on parent work editing
  • somewhat blocked by nurax-pg setup
  • can't replicate in Hyrax
  • registering postgres adapter


Lynette (5449) - collection nesting

  • lots of moving parts
  • exploring current process
  • looking to see if can address the bigger issue of not running the collection indexing unless membership has changed


Shaun (5490) - UI consistency

  • working on ui consistency tickets
  • 2 PRs passing - removal of icons, renaming
  • gonna pick up table sort and column sorting next
  • going to have to sort by solr on server side


Trey (5504) - file derivatives

  • looked some at derivatives issue
  • need parallel of current derivatives job that is valkyrie specific
  • doesn't store derivatives in Fedora currently
  • probably look at tomorrow



Updates Mar 9

Anna (#5461) - batch editing

  • Found a bug that's the cause of the ticket about correctly pulling the permissions that a work is allowed to have - being able to set an embargo/lease. See https://github.com/samvera/hyrax/pull/5514
  • Looking at #5457 - works deposited via proxy getting the right visibility.


Chris (#5463) - missing _pcdm_partials when searching with collections

  • Merged PR resolving the issue of _pcdm collections not being in search results. Ended up cloning a feature test and did some work to make it run against the Valkyrie solr core. Useful patterns here: https://github.com/samvera/hyrax/pull/5515/files
  • Won't be back on the sprint until Friday.


Daniel (#4788) - fileset id not indexed

  • Looking at thumbnail/representative ID indexing. Tricky because derivatives aren't being made. Mostly making sure how the current system works - it's a little convoluted in places.
  • Spent some time trying to solve weird AccessControls::Embargo reloading thing. It seems like ActiveFedora::Relation keeps a cached reference to a class which becomes a different class on reload. Tried to hack it to reload the class, but it got unhappy about wings models.


Eliot (5462) - error editing with parent work

  • Just started looking at #5462 again. Trying to find where in nurax-pg the problem lies - tried to spin up a fresh rails app the same way nurax-pg was and getting the same error. Still tracking down.


Lynette (5157) - collection metadata should be same as in AF

  • (5449) - collection nesting - not worked on today to finish up 5157
  • Update batch delete to use transactions - originally used ActiveFedora to delete.


Shaun (5490) - UI consistency

  • Working on inventorying UI for consistent language/works
  • 3 PRs in. Ping slack or is the github ping better?
    • Slack ping is better.
  • Might need recommendations on a good issue to pick up next.


Trey (5504) - file derivatives

  • Comments in the ticket.

Tao

  • Working on testing.
  • Found many issues with collections. Collection creation/discovery of collections. Those issues are blocking his testing.
  • Lynette is working on nested collections - hopefully there will be work on that soon.
  • Lynette will deploy a new version of nurax-pg with the fixed collection pages.
  • Will Lynette announce new PRs?
    • Lynette pushes with a diff you can browse.

Can someone merge their institution's PRs?

  • Yes

Has Julie approved all the icon 

Shaun needs a new issue

  • Can work on feature tests - can pair with Chris today or Friday.
  • If wants help with the solr backend stuff, can reach out to Lynette for pairing.

Updates Mar 10

Anna (#5457) - proxy deposit visibility always private

  • off this afternoon
  • continuing to work on Issue #5457
  • looked at transactions related to this
  • continuing with this tomorrow

Chris

  • off today
  • Issue #5463 closed by PR #5515 - missing _pcdm_partials when searching with collections

Daniel (#4788) - fileset id not indexed

Eliot (5462) - error editing with parent work OR #5461

Lynette (5157) - collection metadata should be same as in AF

  • PR #5157 - 
  • PR #5511make change set forms allow multiple values for title
  • PR #5518 - adds deprecation notices for transactions/steps that are no longer supported

Shaun (5490) - UI consistency

Tao - testing

Trey (5504) - file derivatives

  • local meeting this afternoon
  • continuing to make progress on derivatives

Updates Mar 11



Week 2


Updates Mar 14



Updates Mar 15



Updates Mar 16



Updates Mar 17



Updates Mar 18

See Retros Dev Congress March 7-18






  • No labels