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

Mostra dias letivos no registro de frequência #148

Open
wants to merge 2 commits into
base: 1.6
Choose a base branch
from

Conversation

gustavomendess
Copy link
Contributor

@gustavomendess gustavomendess commented Jan 9, 2024

Nesta PR foi implementado a amostragem dos dias letivos no registro de frequência.

Hoje em dia, o relatório apenas mostra a quantidade de aulas dadas no período, ou seja, se o professor tiver lecionado apenas 4 aulas, em um período de 10 dias letivos, ele não tem como saber, ele apenas sabe que lançou 4 dias.

A ideia desta PR e o professor também ter clareza de quantos dias ele deveria ter lançado frequência dentro do período selecionado:

Relatório após ajustes:

Emitindo um grande range de datas:
237578378-f7a95303-f33a-4a16-8e4f-5d04c0160846

237578182-a6e2fc23-344b-4168-bed8-e74a0aecf18f

Emitindo um pequeno range de datas:
237578512-7dacb8a6-2734-48e6-b20e-0ddff01c5673

237578538-b1c1a1ca-5459-45ad-9310-627ed7d40d2c

@edersoares
Copy link
Member

@gustavomendess consegue corrigir as imagens deste PR por favor.

@edersoares edersoares changed the base branch from 1.5 to 1.6 June 19, 2024 18:48
Copy link
Member

@edersoares edersoares left a comment

Choose a reason for hiding this comment

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

Será aceito em breve. Obrigado pela contribuição @gustavomendess e @Loginxtech.

@AnaPerola AnaPerola added Ajuda necessária Necessita de colaboração para finalizar waiting and removed ready Ajuda necessária Necessita de colaboração para finalizar labels Jun 20, 2024
@edersoares
Copy link
Member

@gustavomendess consegue atualizar esta branch pra nós por favor.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants