Skip to content
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

Fix ErrorSender among other things #58

Merged
merged 6 commits into from
Feb 17, 2024
Merged

Fix ErrorSender among other things #58

merged 6 commits into from
Feb 17, 2024

Conversation

inetol
Copy link
Member

@inetol inetol commented Feb 17, 2024

En vez de registrar el plugin de forma "normal", se llama directamente a un nuevo método #send() aprovechando "set" que se obtiene en el endpoint.

También se convierten todos los interfaces a type y se mueven a una carpeta dedicada.

@inetol
Copy link
Member Author

inetol commented Feb 17, 2024

@tnfAngel Te invoco para revisarlo ya porque no me termina de gustar tener que arrastrar "set" desde las rutas hasta DocumentHandler con un type any (voy a tener que arreglar un poco el DocumentHandler para simplificarlo así que da un poco igual)

@tnfAngel
Copy link
Member

@tnfAngel Te invoco para revisarlo ya porque no me termina de gustar tener que arrastrar "set" desde las rutas hasta DocumentHandler con un type any (voy a tener que arreglar un poco el DocumentHandler para simplificarlo así que da un poco igual)

según elysia 1.0, se va a incluir error() en el contexto, así que sería más conveniente (cuando la 1.0 esté estable)

@tnfAngel
Copy link
Member

(basándome en lo que he visto en las capturas de https://x.com/saltyAom/status/1758545307640693176?s=20)

Copy link
Member

@tnfAngel tnfAngel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suena bastante bien eliminar constants, así cada cosa está en su clase y todo está más ordenado

Copy link
Member

@tnfAngel tnfAngel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tener una carpeta de tipos me parece buena idea, sobre el any, está bien de momento hasta que salga la 1.0

@inetol
Copy link
Member Author

inetol commented Feb 17, 2024

Suena bastante bien eliminar constants, así cada cosa está en su clase y todo está más ordenado

A ver lo de tenerlo todo centralizado no le veia problema, pero nunca me acuerdo de que puedo seguir usando casi la misma sintaxis que en Java y me gusta bastante mas de este modo

@inetol inetol marked this pull request as ready for review February 17, 2024 21:54
@inetol
Copy link
Member Author

inetol commented Feb 17, 2024

Hay dos errores del tsc que no soy capaz de diagnosticar del Access, estos si llevan ya varias semanas, pero como tampoco parecía que hacían nada los he ido dejando de lado.

Screenshot from 2024-02-17 23-06-51

@tnfAngel
Copy link
Member

tnfAngel commented Feb 17, 2024

Hay dos errores del tsc que no soy capaz de diagnosticar del Access, estos si llevan ya varias semanas, pero como tampoco parecía que hacían nada los he ido dejando de lado.

Screenshot from 2024-02-17 23-06-51

El handleAccess pudiendo devolver un objeto o el propio raw, se podria solucionar con Function Overloading, voy a ello

@tnfAngel
Copy link
Member

(es un error de tipos solo, por eso no hacen nada)

@inetol
Copy link
Member Author

inetol commented Feb 17, 2024

De veras el DocumentHandler es un dolor de huevos que cuando salga la 1.0 voy a rehacerlo entero

@tnfAngel
Copy link
Member

ya debería estar solucionado

@inetol inetol merged commit c5ff036 into dev Feb 17, 2024
6 checks passed
@inetol inetol deleted the fix-errorsender branch February 17, 2024 22:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants