Skip to content

fix: corrigindo erro de exportação#61

Open
katsudon02 wants to merge 2 commits intomainfrom
fix/finishing-export
Open

fix: corrigindo erro de exportação#61
katsudon02 wants to merge 2 commits intomainfrom
fix/finishing-export

Conversation

@katsudon02
Copy link
Copy Markdown
Contributor

Correção do problema de erro de exportação quando semestres planejados ou completos são vazios;
Agora o export usa dados reais.

@katsudon02 katsudon02 requested a review from KakeyaK July 21, 2025 20:44
Teruya11
Teruya11 previously approved these changes Nov 3, 2025
Copy link
Copy Markdown
Contributor

@Teruya11 Teruya11 left a comment

Choose a reason for hiding this comment

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

Pronto para merge

Copy link
Copy Markdown
Contributor

@KakeyaK KakeyaK left a comment

Choose a reason for hiding this comment

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

Melhor limpar os comentários antes de mergear

<td>{{ $subject['name'] }}</td>
<td class="num">{{ $subject['lecture_credits'] }}</td>
<td class="num">{{ $subject['work_credits'] }}</td>
<!-- <td class="num">100</td>
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Apagar comentários

</tr>
@endif
@endforeach
<!-- Totals row removed as per user request -->
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Esse também

<th>Nome da Disciplina</th>
<th>AU</th>
<th>TR</th>
<!-- <th>FREQ</th>
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Esse também

<td>{{ $subject['name'] }}</td>
<td class="num">{{ $subject['lecture_credits'] }}</td>
<td class="num">{{ $subject['work_credits'] }}</td>
<!-- <td class="num">100</td>
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Esse

</tr>
@endif
@endforeach
<!-- Totals row removed as per user request -->
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Esse tb

@KakeyaK KakeyaK requested a review from Teruya11 November 4, 2025 18:02
Removed commented-out columns and totals row from the export template as per user request.

Signed-off-by: Daiki <133552179+Teruya11@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

@Teruya11 Teruya11 left a comment

Choose a reason for hiding this comment

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

Limpo

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