sebbASF commented on pull request #120: URL: https://github.com/apache/whimsy/pull/120#issuecomment-896811346
On Wed, 11 Aug 2021 at 13:56, Shane Curcuru ***@***.***> wrote: > Immaterial of whether or not Infra has a use for some fields - or even if > we have well-understood and thoroughly documented "how all tools use LDAP" > somewhere - I fail to see why we can't merge this code that our volunteer > Secretary(s) use, and that they are both advocating for. > > It doesn't matter if this is the perfect solution, what matters is if it > makes the Secretary job easier without directly harming any future data > organization efforts. Work on #issue 125 doesn't need to gate the Secretary > making progress on their workflow. > > I would really like to see this added so the Secretary test and get back > to work. > The point is that the change does not provide any benefit. We should not be asking the user to provide data that is not needed. > I certainly appreciate sebb's monumental efforts, and the coding portion > of my brain agrees that solving the whole problem in a robust way would be > nice. But blocking progress for other volunteers does not seem appropriate. > AFAICT this is not blocking progress for the Secretary's role. If it is, please could the reason be shared. However I am against unnecessary changes to code, and against unnecessary changes to the information we ask of people who complete the ICLA forms. > — > You are receiving this because you were mentioned. > Reply to this email directly, view it on GitHub > <https://github.com/apache/whimsy/pull/120#issuecomment-896803279>, or > unsubscribe > <https://github.com/notifications/unsubscribe-auth/AD7KQT5KG4QIX7LWZ2I7IXDT4JXQNANCNFSM5A5YANHQ> > . > Triage notifications on the go with GitHub Mobile for iOS > <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> > or Android > <https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email> > . > -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@whimsical.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
