-
Notifications
You must be signed in to change notification settings - Fork 316
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
Reafactoring Tables #821
Comments
Great work here Sandra. Hopefully this proves as well attendance as the breadcrumbs work: certainly this model of openly and clearly sharing the scope of the work and requesting input, seems like a great way to go. Is it worth adding a link to the list of core tables that we are working on too? |
Yep! Many of the core tables have varying difficulties from “seems okay” to “absolutely ridiculous” 😆 so I’m going to take a look and triage them a bit more before adding to the list. |
Hi, I made my first Table OOification in module Policies (GibbonEdu/module-policies#3). |
Awesome, thanks Leo! I'm going to check it out this afternoon 👍 |
I'm gonna steal badges if you guys are ok with that? |
Jump right in @jimbo8098! |
On it! You guys really have been busy! |
Badges has been completed as of yesterday fyi. A PR is awaiting approval for it atm. I did need to update core to add the domain for the module but both PRs are merged with master and v18.0.00 as the case may be. |
Working on Info Grid now |
Awesome, thanks! |
Finished info grid GibbonEdu/module-infoGrid#6 |
Gonna grab house points next. |
Must admit, kind of left this be after my last post due to other projects, however going to work on House Points today. Long time coming! |
Awesome 😃 There were some unreleased changed on the House Points develop branch, so I've noted these in the changelog and pushed a v1.3.02 release. https://github.com/SKuipers/gibbon-house-points |
I'm finishing Credentials. PR in next week! |
Brilliant, thanks!! |
If I'm not mistaken, it seems like the Help Desk module is already converted over to DataTables. I couldn't find any tables tags within that module. |
@raynichc and I completely refactored the whole module, so you'd be right on that one. |
Yes, well spotted, I've checked the Help Desk off on the list 😄 |
FYI I've started work on the Trip Planner module's trips_manage.php page @raynichc |
Hi, finished Alumni. |
Thanks Leo! 👍 |
Questions about the remaining modules:
|
Along with our ongoing refactoring, we've been updating the HTML tables in Gibbon to use a set of PHP classes. This gives us the flexibility to add in better pagination, filtering and sorting, as well as make them mobile-responsive (soon!).
Table OOification is likely the most complicated refactoring so-far. For beginners, see #696 and #820 for a great starting point. Otherwise, give a shout with any questions, since the docs likely don't cover every use case 😅
Info
Here's some info if you'd like to help refactor some tables in Gibbon:
👀 Check out the ooification guide or the codebase for examples
👇 Pick a module that hasn't been started yet
💬 Reply here to let us know which module you're working on. We can also invite you to join the tracker spreadsheet, send an email to: [email protected]
Additional Modules
The text was updated successfully, but these errors were encountered: