Comment 1 Henrik Andersson 2014-11-19 09:55:16 CET

There are also fields that is deprecated, city / location which needs to be handled the correct way eg. do not show them in mail as UNKNOWN, just hide the fields if unset.

Comment 2 Henrik Andersson 2015-01-28 11:33:02 CET

Fixed in commit 29854.

Comment 3 Henrik Andersson 2015-02-02 09:16:33 CET

(In reply to comment #2)
> Fixed in commit 29854.

Typos fixed in commit r29855

Comment 4 Henrik Andersson 2015-02-03 13:57:49 CET

Verified that the mail contained a comment field and no city/location with UNKNOWN values.

Comment 5 Samuel Mannehed 2015-03-31 10:00:32 CEST

The code looks good, and the email seems to be correct:

> Your ThinLinc cluster has the following license files:
> 
> * SUB12-00000467_30815f.zip:user-1-6266.license: 1 concurrent users. Hard limit > of 1 concurrent users. 
>     Customer: testaccount
>     License Number: 6266
>     
>     Comment: 

However, when creating licenses, the comment specified in roundup doesn't make it into the license file, reopening.

Comment 6 Henrik Andersson 2015-03-31 11:09:59 CEST

(In reply to comment #5)
> The code looks good, and the email seems to be correct:
> 
> > Your ThinLinc cluster has the following license files:
> > 
> > * SUB12-00000467_30815f.zip:user-1-6266.license: 1 concurrent users. Hard limit > of 1 concurrent users. 
> >     Customer: testaccount
> >     License Number: 6266
> >     
> >     Comment: 
> 
> However, when creating licenses, the comment specified in roundup doesn't make
> it into the license file, reopening.

Bug in RoundUp, fixed in commit 30194.

Comment 7 Samuel Mannehed 2015-04-01 09:07:56 CEST

Works perfectly now! Closing