liberu-genealogy / genealogy-laravel

Full genealogy application using Laravel 11, PHP 8.3, Filament 3.2 and Livewire 3.5
https://www.liberu.org.uk
MIT License
103 stars 59 forks source link

Sweep: fan chart #148

Closed curtisdelicata closed 4 months ago

curtisdelicata commented 4 months ago

Details

Add a fan chart page under the standard filament 3 location for the data from people / person. Use D3 and convert this project code to use our filament 3,laravel, blade and tailwind

https://github.com/magicsunday/webtrees-fan-chart

Checklist - [X] Create `app/Http/Livewire/FanChart.php` ✓ https://github.com/liberu-genealogy/genealogy-laravel/commit/3ba23fc17338d8dad58f527bf5c47bdcda138b73 [Edit](https://github.com/liberu-genealogy/genealogy-laravel/edit/sweep/fan_chart/app/Http/Livewire/FanChart.php) - [X] Running GitHub Actions for `app/Http/Livewire/FanChart.php` ✓ [Edit](https://github.com/liberu-genealogy/genealogy-laravel/edit/sweep/fan_chart/app/Http/Livewire/FanChart.php) - [X] Create `resources/views/livewire/fan-chart.blade.php` ✓ https://github.com/liberu-genealogy/genealogy-laravel/commit/63191729556a57d86f2a6126ad3d05327cf1aed4 [Edit](https://github.com/liberu-genealogy/genealogy-laravel/edit/sweep/fan_chart/resources/views/livewire/fan-chart.blade.php) - [X] Running GitHub Actions for `resources/views/livewire/fan-chart.blade.php` ✓ [Edit](https://github.com/liberu-genealogy/genealogy-laravel/edit/sweep/fan_chart/resources/views/livewire/fan-chart.blade.php) - [X] Create `public/js/fan-chart.js` ✓ https://github.com/liberu-genealogy/genealogy-laravel/commit/94b449942540375e0706c6b0c9cb40bcbced2a49 [Edit](https://github.com/liberu-genealogy/genealogy-laravel/edit/sweep/fan_chart/public/js/fan-chart.js) - [X] Running GitHub Actions for `public/js/fan-chart.js` ✓ [Edit](https://github.com/liberu-genealogy/genealogy-laravel/edit/sweep/fan_chart/public/js/fan-chart.js) - [X] Modify `routes/web.php` ✓ https://github.com/liberu-genealogy/genealogy-laravel/commit/67f3fe031590ce603dae2d750002f47ab477459f [Edit](https://github.com/liberu-genealogy/genealogy-laravel/edit/sweep/fan_chart/routes/web.php#L1-L1) - [X] Running GitHub Actions for `routes/web.php` ✓ [Edit](https://github.com/liberu-genealogy/genealogy-laravel/edit/sweep/fan_chart/routes/web.php#L1-L1) - [X] Create `app/Http/Controllers/FanChartController.php` ✓ https://github.com/liberu-genealogy/genealogy-laravel/commit/0fe8dcdff8836a9d923ae16757ebddb3c862cbc5 [Edit](https://github.com/liberu-genealogy/genealogy-laravel/edit/sweep/fan_chart/app/Http/Controllers/FanChartController.php) - [X] Running GitHub Actions for `app/Http/Controllers/FanChartController.php` ✓ [Edit](https://github.com/liberu-genealogy/genealogy-laravel/edit/sweep/fan_chart/app/Http/Controllers/FanChartController.php)
sweep-ai[bot] commented 4 months ago

🚀 Here's the PR! #151

See Sweep's progress at the progress dashboard!
Sweep Basic Tier: I'm using GPT-4. You have 4 GPT-4 tickets left for the month and 2 for the day. (tracking ID: 8f951a7812)

For more GPT-4 tickets, visit our payment portal. For a one week free trial, try Sweep Pro (unlimited GPT-4 tickets).

[!TIP] I'll email you at genealogysoftwareuk@gmail.com when I complete this pull request!


Actions (click)

GitHub Actions✓

Here are the GitHub Actions logs prior to making any changes:

Sandbox logs for 9900d12
Checking routes/web.php for syntax errors... ✅ routes/web.php has no syntax errors! 1/1 ✓
Checking routes/web.php for syntax errors...
✅ routes/web.php has no syntax errors!

Sandbox passed on the latest main, so sandbox checks will be enabled for this issue.


Step 1: 🔎 Searching

I found the following snippets in your repository. I will now analyze these snippets and come up with a plan.

Some code snippets I think are relevant in decreasing order of relevance (click to expand). If some file is missing from here, you can mention the path in the ticket description. https://github.com/liberu-genealogy/genealogy-laravel/blob/9900d12421162db31bce44ce51eed120cc68e852/README.md#L31-L63 https://github.com/liberu-genealogy/genealogy-laravel/blob/9900d12421162db31bce44ce51eed120cc68e852/CONTRIBUTING.md#L1-L10
I also found the following external resources that might be helpful: **Summaries of links found in the content:** https://github.com/magicsunday/webtrees-fan-chart: The GitHub page is for a module called "webtrees-fan-chart" that provides an SVG-based ancestor fan chart for the webtrees genealogy application. The page includes installation instructions for the module, which can be done manually or using Composer or Git. The module can be activated within webtrees' control panel and offers configuration options to adjust the layout of the charts. The page also provides code snippets for building and updating the JavaScript, running tests, and analyzing code. The module is licensed under GPL-3.0 and has received 54 stars and 17 forks on GitHub. The user can find the code and more information about the module at the provided GitHub link.

Step 2: ⌨️ Coding

Ran GitHub Actions for 3ba23fc17338d8dad58f527bf5c47bdcda138b73:

Ran GitHub Actions for 63191729556a57d86f2a6126ad3d05327cf1aed4:

Ran GitHub Actions for 94b449942540375e0706c6b0c9cb40bcbced2a49:

--- 
+++ 
@@ -15,4 +15,6 @@

 Route::get('/', function () {
     return view('layouts.home');
+
+Route::get('/fan-chart', 'App\Http\Controllers\FanChartController@show');
 });

Ran GitHub Actions for 67f3fe031590ce603dae2d750002f47ab477459f:

Ran GitHub Actions for 0fe8dcdff8836a9d923ae16757ebddb3c862cbc5:


Step 3: 🔁 Code Review

I have finished reviewing the code for completeness. I did not find errors for sweep/fan_chart.


🎉 Latest improvements to Sweep:
  • New dashboard launched for real-time tracking of Sweep issues, covering all stages from search to coding.
  • Integration of OpenAI's latest Assistant API for more efficient and reliable code planning and editing, improving speed by 3x.
  • Use the GitHub issues extension for creating Sweep issues directly from your editor.

💡 To recreate the pull request edit the issue title or description. To tweak the pull request, leave a comment on the pull request.Something wrong? Let us know.

This is an automated message generated by Sweep AI.