Add Resource Geogebra #131

marisalerno · 2021-07-04T14:35:16Z

Add Geogebra Resource

ignaciojonas

Creo que el resto esta bien. Solo ese cambio para que puedan pasar los tests.

ignaciojonas · 2021-07-19T15:57:45Z

database/factories/ResourceFactory.php

@@ -18,6 +18,7 @@
    return [
        'name' => $faker->word,
        'type' => $faker->text,
-        'link' => $faker->url
+        'link' => $faker->url,
+        'geogebra' => $faker->GGBApplet


Esto no funciona, porque no existe en Faker, que genere un id de Geogebra. Lo que se puede hacer aca es poner o un ID fijo de un recurso de geogebra que conozcas. Porque esto nos esta rompiendo los tests.

ignaciojonas · 2021-07-19T16:20:21Z

resources/views/Resource/show.blade.php

+            @if ($resource->type == "Geogebra")
+              <div id="ggb-element"></div>
+              <script>
+                var ggbApp = new GGBApplet({"appName": "graphing", "width": 800, "height": 600, "showToolBar": true, "showAlgebraInput": true, "showMenuBar": true, "material_id":"{$resource->geogebra_id)}" }, true);


Aca en appName, vamos a tener que poder modificarlo. Si miras en la ayuda https://wiki.geogebra.org/es/Referencia:Incrustaci%C3%B3n_de_Aplicaciones_GeoGebra dependiendo el recurso que uses, si es uno 3d o uno de graficos o uno de geometria, hay que configurar la app bien. Entonces, deberiamos modificar el recurso para que cuando se selecciona Geogebra, le podamos seleccionar el tipo de app que queremos: graphing a geometry o 3d

ignaciojonas · 2021-07-19T16:21:44Z

app/Resource.php

@@ -6,5 +6,5 @@

 class Resource extends Model
 {
-      protected $fillable = ['name','type', 'path', 'link'];
+      protected $fillable = ['name','type', 'path', 'link', 'geogebra_id'];


Lo que no veo es la migracion que agrega esa columna. Deberiamos, como te comente en otro comentario, agregar ademas una geogebra_app_type para que se pueda guardar el ID del recurso con el tipo de app para mostrarlo bien.

ignaciojonas

@marisalerno Ahi te agregue unos comentarios para guiarte en los pasos que tenes que seguir para hacer anda el recurso de geogebra.
Estas encaminada, pero hay algunos errores.
Fijate bien los comentarios y cualquier cosa me escribir.

Los pasos generales serían:

Creas una migracion que agrega 2 campos geogebra_type geogebra_resource
Actualizas las seeds y los factories
Corres las migraciones
Actualizas los templates para que muestren correctamente los campos de la base
Actualizas los modelos para que acepten los nuevos campos

Una vez que lo tengas andando vemos de poder meterle algunos tests.

ignaciojonas · 2021-08-13T02:10:28Z

app/Http/Controllers/ResourceController.php

@@ -9,7 +9,8 @@
 class ResourceController extends Controller
 {

-    public $types = ['Audio','Video', 'Imagen', 'Documento','Link'];
+    public $types = ['Audio','Video', 'Imagen', 'Documento','Link', 'Geogebra'];
+    public $geotypes = ['graphing', 'geometry', '3d'];


Muy bien esto!

ignaciojonas · 2021-08-13T02:12:13Z

app/Resource.php

@@ -6,5 +6,5 @@

 class Resource extends Model
 {
-      protected $fillable = ['name','type', 'path', 'link'];
+      protected $fillable = ['name','type', 'path', 'link', 'geotype'];


ponele en lugar de geotype, geogebra_type. Para ponerlo en el modelo tenes que crear una migracion. Que va a modificar la tabla Resource y agergar 2 campos.
geogebra_type (este es el que ya tenes)
geogebra_code (est ees el codigo que tiene que poner el docente para que se vea)

en esta linea que esta el comentario tenes que agregar tambien geogebra_code

ignaciojonas · 2021-08-13T02:14:09Z

database/factories/ResourceFactory.php

@@ -18,6 +18,7 @@
    return [
        'name' => $faker->word,
        'type' => $faker->text,
-        'link' => $faker->url
+        'link' => $faker->url,
+        'geotype' => 'ZT6xkDYM'


Cambiar por geogebra_type. Aca ese codigo esta mal, lo que es el tipo es 3d, graphing, geometry. osea podes poner cualquiera de esos.
despues pones otro geogebra_code con ese valor.

Justo para ese codigotenes que poner graphing

ignaciojonas · 2021-08-13T02:15:46Z

database/migrations/2018_02_16_195855_create_resources_table.php

@@ -19,6 +19,7 @@ public function up()
            $table->string('type');
            $table->string('path')->nullable();
            $table->string('link')->nullable();
+            $table->string('geotype')->nullable();


esto no lo tenes que hacer aca a pelo, tenes que hacer una migracion nueva
php artisan make:migration add_geogebra_to_resources_table
y adento de esa creas las columnas (hay un ejemplo en el codigo add_record....

Una vez que completaste el up y down de esa migracion ejecutas las migraciones
php artisan migrate

ignaciojonas · 2021-08-13T02:16:18Z

database/seeds/ResourcesTableSeeder.php

@@ -17,6 +17,7 @@ public function run()
      factory(App\Resource::class, 1)->create(['name'=>'presentación', 'type'=>'presentación de diapositivas']);
      factory(App\Resource::class, 1)->create(['name'=>'hipervínculo', 'type'=>'hipervínculo']);
      factory(App\Resource::class, 1)->create(['name'=>'imagen', 'type'=>'imagen']);
+      factory(App\Resource::class, 1)->create(['name'=>'ggb-element', 'type'=>'app geogebra']);


Name puede ser cualqueir cosa
Type deberia ser Geogebra que es el Type que le ponemos.

ignaciojonas · 2021-08-13T02:16:39Z

resources/views/Resource/create.blade.php

@@ -2,6 +2,8 @@

 @section('pagespecificscripts')
  <script src="{{ asset('js/resources.js') }}"></script>
+  <script src="https://www.geogebra.org/apps/deployggb.js"></script>


aca no lo necesitamos a esto.

ignaciojonas · 2021-08-13T02:19:16Z

resources/views/Resource/show.blade.php

+              <script>
+                var ggbApp = new GGBApplet({"appName": "{$resource->geotype}", "width": 800, "height": 600, "showToolBar": true, "showAlgebraInput": true, "showMenuBar": true, "material_id":"{$resource->name)}" }, true);
+                window.addEventListener("load", function() {
+                  ggbApp.inject('ggb-element');
+                });
+              </script>


Todo esto lo tenes que pasar a la seccion de arriba donde estan los scripts. (linea 5)
Vas a tener que agregar el if de @if ($resource->type == "Geogebra") ahi tambien para que se complete bien todos los datos.
En la parte de appName tenes que poner "{{$resource->geogebra_type}}" (lleva dobe llave, no una sola como tenes ahi) y en la de material_id, el otro campo que create con el codigo

ignaciojonas · 2021-08-13T02:20:10Z

resources/views/Resource/create.blade.php

+            <div class="form-group ggb-element">
+                <label for="geotype" class="col-md-2 control-label"><p>Seleccione tipo de elemento Geogebra</p></label>
+                <div class="col-md-6">
+                      <select class="form-control" name="geotype" id= "geotype">


aca el name y id van a ser geogebra_type

ignaciojonas · 2021-08-13T02:20:42Z

resources/views/Resource/create.blade.php

+                        @endforeach
+                      </select>
+                </div>
+            </div>


tenes que agregar ademas un input para el otro campo que creaste geogebra_code. Es un input de tipo text

ignaciojonas · 2021-08-13T02:21:56Z

resources/views/Resource/update.blade.php

+          <div class="form-group ggb-element">
+              <label for="ggb-element" class="col-md-2 control-label"><p>Geogebra</p></label>
+              <div class="col-md-6">
+                  <input id="ggb-element" type="text" class="form-control" value="{{$resource->geogebra_id}}" name="ggb-element" autofocus>
+              </div>
+          </div>
+


Esto te tiene que quedar igual a lo que pusiste en el tempate de show, la diferencia es que tenes que agregar en el select, que se seleccione el geogebra_type que esta grabado en la base y en el geogebra_code, se complete el valor de la base tambien.

Add Resource Geogebra

0051858

ignaciojonas requested changes Jul 19, 2021

View reviewed changes

ignaciojonas reviewed Jul 19, 2021

View reviewed changes

possible changes to make geogebra work

b29ebc5

ignaciojonas requested changes Aug 13, 2021

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add Resource Geogebra #131

Add Resource Geogebra #131

marisalerno commented Jul 4, 2021

ignaciojonas left a comment

ignaciojonas Jul 19, 2021

ignaciojonas Jul 19, 2021

ignaciojonas Jul 19, 2021

ignaciojonas left a comment

ignaciojonas Aug 13, 2021

ignaciojonas Aug 13, 2021

ignaciojonas Aug 13, 2021

ignaciojonas Aug 13, 2021

ignaciojonas Aug 13, 2021

ignaciojonas Aug 13, 2021

ignaciojonas Aug 13, 2021

ignaciojonas Aug 13, 2021

ignaciojonas Aug 13, 2021

ignaciojonas Aug 13, 2021

Add Resource Geogebra #131

Are you sure you want to change the base?

Add Resource Geogebra #131

Conversation

marisalerno commented Jul 4, 2021

ignaciojonas left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

ignaciojonas left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment