PPS Feli pasada a SEDICI 9.3#15
Conversation
…ificaciones al dropdown de filtros
…cción que no sea la actual
…to en cargar y los espacios extras en las fechas., Elimino código obsoleto
…una vez que se reposiciona
…e pasar la selección al campo focuseado, elimino accesos rápidos obsoletos
…impieza general del texto. Modifico otros para abarcar más casos
… campo elegido previamente) y se muestra el resultado en el previsualizador
…sí no está más la opción de agregar/reemplzar/concatenar.
Repetibles se agregan siempre. No repetibles se sobreescriben siempre.
Hago que la previsualización ocupe el mayor alto posible.
…ión de filtros sobre inputs
…dos al texto seleccionado
…resoluciones chicas
…icador de filtros
Hago la prueba de resaltar los resultados obtenidos del split para personas
| transformedText = filterTransformer.removeReferences(text); | ||
| break; | ||
| case 'transformPersons': | ||
| console.log('persons después', filterTransformer.transformPersons(text)); |
There was a problem hiding this comment.
Ese archivo se puede borrar por completo, es un html de pruebas.
La carpeta que lo contiene tiene una "z" de más; hay que ver de agregarla al src
| <p id="resultText"></p> | ||
| </div> | ||
| </body> | ||
| </html> |
There was a problem hiding this comment.
Si borro el .html, paso el .js a /src/app/submission/utils o creo /src/themes/custom/app/submission/utils?
Sería necesario dejarlo como .js o directamente lo paso a .ts como el resto del código?
There was a problem hiding this comment.
Se eliminó el .html.
Queda mover el otro archivo
|
|
||
| private isSubmissionUrl(url: string): boolean { | ||
| return url.startsWith('/workspaceitems') && url.endsWith('/edit'); | ||
| } |
There was a problem hiding this comment.
En general los cambios son los mismos del header.component.ts
| } | ||
|
|
||
| export class DateSplitter { | ||
| private readonly monthNames: Record<string, string> = { |
There was a problem hiding this comment.
Quiza se pueda usar algo como el modulo de translate para no tener que armar esto para cada idoma?
|
|
||
| private interceptFormOperationChanges(): void { | ||
| const originalDispatch = this.formOperationsService.dispatchOperationsFromChangeEvent; | ||
| this.formOperationsService.dispatchOperationsFromChangeEvent = |
There was a problem hiding this comment.
Este service creo que es un Singleton. Cada vez que el usuario entray sale de la pantalla se ejecuta este método sobreescribiendo la misma instancia del service y, por lo tanto, agregando los 500ms nuevamente (500ms, 1000ms, ...). Para que se reinicie tengo entendido que hay que agregarlo al ngOnDestroy.
if (originalDispactch) then distroy
…o se perdió en una migración)
There was a problem hiding this comment.
Ya lo comente en otro archivo pero en este es mas facil, si se prepara para comitear en DSpace hay que cambiarlo para que use el translate en la description
| const match = input.id.match(/(dc|sedici|mods|thesis).*/); | ||
| componentRef.instance.inputID = match ? match[0] : input.id; | ||
|
|
||
| componentRef.instance.filterApplied.subscribe((filter: string) => { |
There was a problem hiding this comment.
Habría que hacer unsuscribe en el onDestroy?
| } | ||
| }); | ||
|
|
||
| const formContainer = document.body; |
There was a problem hiding this comment.
Es necesario observar absolutamente todo el body? o podría ser solamente la parte del form con los botones y eso?
| @if (!item.isWithdrawn || (isAdmin$|async)) { | ||
| <ds-sedici-certificate [object]="item" [viewMode]="viewMode"></ds-sedici-certificate> | ||
| } | ||
| <!--<ds-listable-object-component-loader *ngIf="!item.isWithdrawn || (isAdmin$|async)" [object]="item" [viewMode]="viewMode"></ds-listable-object-component-loader>--> |
| <ds-dso-edit-menu class="edit-menu"></ds-dso-edit-menu> | ||
| </div> | ||
| <h2 class="item-title me-auto"> | ||
| <span [innerHTML]="object.firstMetadataValue('dc.title')"></span>@if (object.firstMetadataValue('sedici.title.subtitle') && object.firstMetadataValue('dc.title.alternative')) { |
There was a problem hiding this comment.
el if creo que va en la linea de abajo (supongo q es algo visual nomas)
| } | ||
| // if (!this.active && isNotNull(activeSectionId)) { | ||
| // this.submissionService.dispatchSave(this.submissionId); | ||
| // } |
There was a problem hiding this comment.
El problema es el siguiente: el formulario se guarda solo al acceder a campos de una nueva sección del formulario; esto rompe la carga de palabras clave, por ejemplo, porque se carga la primera --> se detecta el cambio --> se guarda el formulario --> ese proceso genera como una "pausa" que hace fallar la carga de las siguientes palabras clave.
Habría que buscar una forma más elegante de resolverlo.
… botón (fieldButtonMap y createButtonForField)
…creación de componentes dinámicos
3 participants
PR para hacer una revisión de código y componentes modificados, de una forma más sencilla.
No se espera hacer el merge directamente, sino seguir trabajando desde la rama de la pps hasta tener un código un poco más pulido.
Entre los cambios aparecen componentes que no tienen que ver con la PPS: algunos tuvieron actualizaciones en su sintaxis de Angular, otros son mejoras que fueron surgiendo, y otros eran cambios que ya se habían aplicado pero se borraron en alguna actualización.