[Xerte-dev] Re: Released several fixes to 3.01
Fay Cross
Fay.Cross at nottingham.ac.uk
Thu Sep 10 12:50:57 BST 2015
Thanks Tom. I created a new hotfix branch yesterday with the fix in. I've tested it a fair bit and it's on our installation and seems to be ok. I'll leave it until next week before merging it just in case.
Fay
From: xerte-dev-bounces at lists.nottingham.ac.uk [mailto:xerte-dev-bounces at lists.nottingham.ac.uk] On Behalf Of Tom Reijnders
Sent: 09 September 2015 15:40
To: For Xerte technical developers
Subject: [Xerte-dev] Re: Released several fixes to 3.01
Yes, that's really the best. If you fix it in devlop, and also separately in master, we could end up with conflicts when trying to merge develop into master again.
If it's really a 1 line fix, you could also consider to fix it in master, and merge into develop.
You generally want to do the fix on a hotfix branch (from master), push it to github, ask others to review, cq. test it, and than merge into master and develop.
Tom
On Wed, 9 Sep 2015 14:16:41 +0000 Fay Cross <Fay.Cross at nottingham.ac.uk<mailto:Fay.Cross at nottingham.ac.uk>> wrote
I'm pretty sure there's a problem in this line breaks hotfix code. From what I can work out toolbox.js goes through every page's attributes and if the XML is from the old editor it calls addLineBreaks function to look at whether the attribute text needs to be adjusted so line breaks aren't lost. This works ok for text stored in an attribute but not where the text is in the CDATA (e.g. text, bullets, advanced image pages). I don't think text in here is being looked at.
I can fix this - should I do it in another hotfix then? And then merge into master like below? It will just be a change to 1 file. I just wanted to check as I've only made changes in develop before.
Thanks
From: xerte-dev-bounces at lists.nottingham.ac.uk<mailto:xerte-dev-bounces at lists.nottingham.ac.uk> [mailto:xerte-dev-bounces at lists.nottingham.ac.uk] On Behalf Of Tom Reijnders
Sent: 24 August 2015 09:32
To: For Xerte technical developers
Subject: [Xerte-dev] Released several fixes to 3.01
L.S.,
Over the weekend, I merged the hotfix branch (linebreaks) and several fixes done on develop into master (3.0) and set a annotated tag v3.01. Effectively this means that vesion.txt now shows 3.0 (v3.01...).
Everything is also merged into develop.
As a reminder, if you know a fix has to end up in the current release, you should:
1. create a hotfix branch of master
2. implement the fix
3. test test test
4. merge the hotfix branch into master and into develop
Only do this for fatal bugs.
Regards,
Tom
--
--
Tom Reijnders
TOR Informatica
Chopinlaan 27
5242HM Rosmalen
Tel: 073 5226191
Fax: 073 5226196
This message and any attachment are intended solely for the addressee
and may contain confidential information. If you have received this
message in error, please send it back to me, and immediately delete it.
Please do not use, copy or disclose the information contained in this
message or in any attachment. Any views or opinions expressed by the
author of this email do not necessarily reflect the views of the
University of Nottingham.
This message has been checked for viruses but the contents of an
attachment may still contain software viruses which could damage your
computer system, you are advised to perform your own checks. Email
communications with the University of Nottingham may be monitored as
permitted by UK legislation.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.nottingham.ac.uk/pipermail/xerte-dev/attachments/20150910/c7f6cede/attachment.html>
More information about the Xerte-dev
mailing list