by Luc Saffre · in Developers · assigned to Luc Saffre

Résumé

Concepts sous-jacents

Commentaires

• 1 année, 5 mois 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 année, 5 mois 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 année, 5 mois 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 année, 5 mois 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 année, 5 mois 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 année, 5 mois 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 année, 5 mois 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 année, 5 mois 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 année, 5 mois 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 année, 5 mois 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).