by Luc Saffre · in Developers · assigned to Luc Saffre

Kurzbeschreibung

Kinder

Kommentare

• 1 Jahr, 5 Monate ago von Luc Saffre about #6119 (Don't show a Duplicate button on CheckPeppol) : The LinksByHuman panel is *also* visible for 220 when I switch the default_ui of weleup1 to ExtJS. Which means that if this is a bug, then it has been there already before my changes. So I will accept the new visibilities (adapt the doctest) and not worry further right now. It also means that we mig...
• 1 Jahr, 5 Monate ago von Luc Saffre about #6119 (Don't show a Duplicate button on CheckPeppol) : One of the differences in visibility is the LinksByHuman panel, which is in the "Human links" tab of the ClientDetail in gerd (weleup). The LinksByHuman actor has required_roles = dd.login_required(ContactsUser), and user type 220 does indeed not have this role (as shown here), so these users should...
• 1 Jahr, 5 Monate ago von Luc Saffre about #6119 (Don't show a Duplicate button on CheckPeppol) : I'm trying to understand the last two doctest failures in welfare. Some panels in ClientDetail have their visibility changed.
• 1 Jahr, 5 Monate ago von Luc Saffre about #6119 (Don't show a Duplicate button on CheckPeppol) (2 Antworten) : I'm still having the following failure in docs/specs/weleup/eupen.rst:
  638 >>> print(analyzer.show_window_fields()) Differences (unified diff with -expected +actual): @@ -71,5 +71,5 @@ - checkdata.Checkers.detail : value, text - checkdata.Messages.detail : checker, owner, message, user...
• 1 Jahr, 5 Monate ago von Luc Saffre about #6119 (Don't show a Duplicate button on CheckPeppol) : And I'm still not done with it: the welfare test suite reveals some unwanted side effects regarding visibility of panels, which I must fix before I can publish my work.
• 1 Jahr, 5 Monate ago von Luc Saffre about #6119 (Don't show a Duplicate button on CheckPeppol) : As another side effect I moved things around for #6096, which makes the changeset quite bit and which was actually not needed for this ticket but at some moment I thought it did.
• 1 Jahr, 5 Monate ago von Luc Saffre about #6119 (Don't show a Duplicate button on CheckPeppol) : Yet another side effect after fixing #5739: pm demotest in min9 raises "Item memo_MentionsByTarget_htmlbox2129 of tabbed LayoutHandle for lino_noi.lib.users.ui.UserDetail on lino.modlib.users.ui.Users has no label!"
  
• 1 Jahr, 5 Monate ago von Luc Saffre about #6119 (Don't show a Duplicate button on CheckPeppol) : He he, indeed it is touching the #5739 (Oops, get_atomizer(...) returned None) issue.
• 1 Jahr, 5 Monate ago von Luc Saffre about #6119 (Don't show a Duplicate button on CheckPeppol) : I think I fixed this, but I don't yet dare to push my changes because I made some dangerous core changes, and the Read eID card action in avanti1 is failing. I'm not sure whether this failure is caused by my changes or was it there already before and just had slipped through the test suite.
• 1 Jahr, 5 Monate ago von Luc Saffre about #6119 (Don't show a Duplicate button on CheckPeppol) : Created new Ticket #6119 (Don't show a Duplicate button on CheckPeppol).