We have moved to GitHub Issues
Created by Matt Brailsford 18 Dec 2015, 16:06:48 Updated by Robert Copilau 12 Sep 2017, 06:27:54Tags: Up For Grabs PR
Is duplicated by: U4-7649
Is duplicated by: U4-7798
Relates to: U4-9143
Parent for: deploy-388
Parent for: COU-616
Subtask of: UAASSCRUM-1073
Once you've made a doctype folder, there doesn't seem to be a way to rename it
We'll get to this after the initial release
I'm grabbing this for the hackathon at uk fest.
PR'd this and U4-9143 under the same PR starting with U4-7580
@lars-erik Awesome #H5YR
Would be great to have this reviewed for an upcoming patch release - given the number of votes and a PR being available.
definitely, i'll do it today
NOTE: We also need to make sure that this works with Courier + Deploy ... once we get feedback on the PR we'll see what is involved with the above. Might not make 7.7.0 if there's a bit of work there.
After some testing, even though the feature works as intended, it does not work with Courier or Deploy. It seams that whenever the folder is renamed, it does not trigger the event that makes Deploy/Courier save/update the revision file.
I will try and get to the bottom of this issue at least for Deploy, as Claus put it, I'll probably brake my neck if I would try to fix it for Courier - but I'll try anyway :D
Will that be alright @Shandem? Does this need to be a separate task?
@Robrobertcopilau ahh yes of course... @claus did remind me of this. In this case we need 2 more tasks: 1 for deploy and 1 for courier, however seeing as though making those updates may take some time, we therefore cannot release this feature as part of 7.6.6 or 7.7.0 it will have to wait for 7.7.x (or potentially 7.6.7 if we make that release) when we are done updating Courier and Deploy and everything is fully tested. If you can create those two tasks and assign them to next sprint then we can start working on them then. We'll also transition this task over to next sprint as well Thanks!!
Changes: *Added missing event to trigger deploy/courier. Has only been tested with deploy but it will probably work with courier as well.
The problem is that Courier will not detect any changes done to the folder and the nothing will happen. A solution is to rename the folder/s manually.
Unless @Shandem or @claus has any objections this can be merged.
I guess the only thing missing now is Data Type Folder renames ;)
I might make it to the hackathon at UK fest. We'll see. ;)
I think @robertcopilau might beat you to it, we'll see :)
Go ahead! :) \o/
Don't mind if I do :D
Priority: Up for grabs
Backwards Compatible: True
Fix Submitted: Pull request
Affected versions: 7.4.0
Due in version: 7.7.0
Sprint: Sprint 67
Story Points: 1