atom icon indicating copy to clipboard operation
atom copied to clipboard

Add authority record rename module

Open mInnes-archives opened this issue 2 years ago • 2 comments

PR adds a rename module for use with authority records.

It largely duplicates the design and functionality of the existing rename module used for descriptions, minus aspects not relevant such as the repository sidebar and functions for updating attached finding aids.

I've also moved the authority record action buttons out of the ISAAR plugin for more general use.

Should be able to close issue #1006 if approved for merge.

mInnes-archives avatar Oct 13 '22 23:10 mInnes-archives

Hi @mInnes-archives !

Wow, thank you so much for this contribution! As an FYI, it might be a little bit before a developer at Artefactual is available to review this work.

As you may know, we're currently in the process of doing some reorganization at Artefactual, and responsibility for maintaining AtoM is going to be passed to a yet-to-be-formed team. Currently an ad-hoc team is focused on finalizing the 2.7 release, and we then hope to address ongoing project ownership questions in the new year. Internally we have already hit feature freeze for 2.7, so it likely won't be until after the 2.7 public release that someone can review this work and consider it for inclusion... but I'll make sure that whoever is picking up the reins for 2.8 is aware of this work and can do a proper assessment and code review for you!

Until then, thanks in advance for your patience and understanding - and thank you again for contributing this work to the AtoM project!

fiver-watson avatar Oct 14 '22 13:10 fiver-watson

All good, thanks for the update.

mInnes-archives avatar Oct 14 '22 15:10 mInnes-archives

Hi @mInnes-archives! Apologies for not getting to this sooner! I'm one of the maintainers from the new team, and I'm looking into getting this into the next release! Thanks so much for your contribution, this would be a really good feature to have!

I do have some feedback that I wanted to share. Let us know if you'd have the time/capacity to make these changes! If not, we're happy to make them and accept this PR. Thanks again!

anvit avatar Apr 17 '23 19:04 anvit

Hi @anvit. I don't see an issue getting to this in the next couple weeks. Is that an acceptable timeframe?

mInnes-archives avatar Apr 17 '23 20:04 mInnes-archives

Yes, in the next couple weeks would be great! Thanks!

anvit avatar Apr 17 '23 20:04 anvit

@anvit should be good to go now.

mInnes-archives avatar May 01 '23 02:05 mInnes-archives