by Luc Saffre · in Developers · assigned to Luc Saffre

Summary

Children

Comments

• 1 year, 5 months ago by 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 year, 5 months ago by 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 year, 5 months ago by 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 year, 5 months ago by Luc Saffre about #6119 (Don't show a Duplicate button on CheckPeppol) (2 replies) : 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 year, 5 months ago by 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 year, 5 months ago by 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 year, 5 months ago by 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 year, 5 months ago by 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 year, 5 months ago by 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 year, 5 months ago by Luc Saffre about #6119 (Don't show a Duplicate button on CheckPeppol) : Created new Ticket #6119 (Don't show a Duplicate button on CheckPeppol).