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

Conexion informar pago #126

Merged
merged 21 commits into from
Sep 25, 2024
Merged

Conexion informar pago #126

merged 21 commits into from
Sep 25, 2024

Conversation

yoelmarain
Copy link
Contributor

No description provided.

yoelmarain and others added 21 commits September 23, 2024 09:55
- Add a new input field for comments in the payment form
- Update the backend post request to include the comments field
- Render the comments field in the payment table

fix: Update payment table calculations

- Update the calculation of the amount owed for each payment
- Consider the amount paid, the amount informed, and the current amount for each payment
- Display the correct amount owed in the payment table

fix: Format payment amount and add thousands separator

- Format the payment amount to include a thousands separator
- Update the payment table to display the formatted amount

fix: Update payment status badge colors

- Update the colors of the payment status badges in the payment table
- Use red for "ADEUDADO" status, yellow for "INFORMADO" status, and green for "PAGADO" status

fix: Update payment form layout

- Adjust the layout of the payment form to include the new comments field
- Update the styling of the form elements

fix: Update payment status card layout

- Adjust the layout of the payment status card to include the current date
- Update the styling of the card

fix: Fetch payments for selected student

- Update the fetch request to retrieve the payments for the selected student
- Display the fetched payments in the payment table
- Added FetchGetCuotas function to fetch cuotas for a specific alumno.
- Modified the FetchGetCuotas function to use a hardcoded dni value for testing purposes.

feat: Add payment comments field and implement payment submission

- Added payment comments field to the payment form.
- Implemented payment submission functionality.

feat: Add payment comments field

- Added payment comments field to the payment form.

fix: Update variable name in TablaPagos component

- Updated variable name from cuota.montoactual to cuota.montoActual in the TablaPagos component.

feat: Add Compromiso API functions

- Added FetchCompromisos function to fetch compromisos de pago for a specific alumno.
- Added FetchUltimoCompromiso function to fetch the last compromiso de pago.
- Added FirmarCompromiso function to sign the compromiso de pago.
- In Compromiso.ts, updated the fetch URL to use the new API endpoint for fetching firmas-de-alumno.
- Removed the unused code for getting the token and dni from cookies.

Improve UI and functionality in CompromisoDePago.tsx

- Added a conditional rendering to display a message when the current compromiso de pago is already signed.
- Adjusted the size of the PDF iframe and added a button to sign the compromiso.
- Updated the table headers and added a heading for the history of signed compromisos.
The code changes in `Compromiso.ts` and `Pagos.ts` fix an issue with fetching and signing commitments. The previous code was not correctly retrieving the user's DNI from the cookies, resulting in incorrect data being fetched and signed. The updated code now uses a hardcoded DNI value for testing purposes. Additionally, the `Pagos.ts` file now includes a new parameter `nro_transferencia` for sending the transfer number along with the payment information.

This commit also includes commented out code in `TablaPagos.tsx` that was causing a page reload. The code was commented out as it was not necessary and caused unnecessary page refreshes.
- Updated the `TablaPagos` component to format currency values using the `Intl.NumberFormat` API for better readability.
- Added logging statements to `FetchPostPago` function to debug form data and parameters.
- Improved the display of payment details in the `TablaCuotas` component.
@yoelmarain yoelmarain merged commit 6eeecd3 into dev Sep 25, 2024
1 check failed
@yoelmarain yoelmarain deleted the Conexion-Informar-Pago branch September 25, 2024 16:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants