ShaneCurcuru commented on pull request #120: URL: https://github.com/apache/whimsy/pull/120#issuecomment-896803279
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. 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. -- 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
