-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Possibly use byu.hr.manage_apps
as a module
#3
Comments
Comparing
This would reduce the size of boilerplate code by 19 lines as those rules can be omitted from the boilerplate, along with the binding of the The above does not require usage as a module. Furthermore, the convenience functions could be provided by the module, saving another 10 lines of boilerplate. A call to these functions would have to pass along the RID, as apps are keyed by the RID. Assuming
the calls would look like:
|
A generated boilerplate would look like
|
Refactoring the HTML generating boilerplate, providing an
the boilerplate could be further simplified to these 16 lines, assuming a RID of
|
See issue #90 of that repo.
The text was updated successfully, but these errors were encountered: