bdemers commented on PR #909: URL: https://github.com/apache/directory-scimple/pull/909#issuecomment-3285858859
@CS-Birb I took a quick pass at replacing the attribute args with a `ScimRequestContext` (just the `create` method at first to get a feel for how it would look, but the rest of the methods would need to be implemented before merging) I think this could work, but I don't like my fall back strategy (default method, calls the old method). Implementors would need to implement the new method, and the old, which feel wrong. Any thoughts? -- 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...@directory.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@directory.apache.org For additional commands, e-mail: dev-h...@directory.apache.org
