student council page

[Soumil221]

No description provided.

[Soumil221]

@Aryawart-kathpal @Debatreya this has my changes

[Copilot]

Pull request overview

This pull request adds a new Student Council page to the student activities section of the website. The page displays different categories of Student Council members (Institute Functionaries, Core Committee, Nominated Students, and Student Representatives) with their details fetched from the database.

Changes:

• Added Student Council translations for English and Hindi
• Created a new Student Council page component that fetches and displays member information organized by category
• Integrated the translations into the i18n system

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 8 comments.

Show a summary per file

File	Description
i18n/translations.ts	Added StudentCouncilTranslations type and export, reorganized import order
i18n/translate/student-council.ts	New translation file with English and Hindi translations for Student Council page
i18n/translate/index.ts	Exported student-council translations
i18n/hi.ts	Added Hindi Student Council translations to the main translations object
i18n/en.ts	Added English Student Council translations to the main translations object
app/[locale]/student-activities/student-council/page.tsx	New page component displaying Student Council members with database queries and multiple display sections

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Missing revalidation configuration. Pages with database-backed server components in this directory typically set export const revalidate = 300; at the top to revalidate every 5 minutes. This page makes multiple database queries and should include this configuration for consistent caching behavior.

[Copilot]

Hardcoded year value 2026 will become outdated. The year calculation logic 2026 - academicData.batch should use the current year dynamically (e.g., new Date().getFullYear() - academicData.batch) to ensure the calculation remains accurate over time.

Suggested change

	? 2026 - academicData.batch
	? new Date().getFullYear() - academicData.batch

[Copilot]

The image source path appears incorrect for the Student Council page. The path "student-activities/thought-lab/welcome-bk-shivani.jpeg" references a Thought Lab image, but should reference an appropriate Student Council image instead.

Suggested change

	src="student-activities/thought-lab/welcome-bk-shivani.jpeg"
	src="student-activities/student-council/student-council-header.jpeg"

[Copilot]

The getCount prop is passed as Promise.resolve([]) which always returns an empty array. This will likely cause pagination issues or incorrect behavior. Either pass the correct count query or omit the prop if pagination is not needed for this table.

Suggested change

getCount={Promise.resolve([])}

[Debatreya]

@Soumil221 check this

[Copilot]

Unnecessary space in template literal. The expression ${year} adds a leading space before the year value. This should be ${year} without the leading space, similar to how it's handled for other fields.

Suggested change

	year: year ? ` ${year}` : '-',
	year: year ? `${year}` : '-',

[Debatreya]

@Soumil221 check this

[Copilot]

Inconsistent notification category. The NotificationsPanel uses category="student-activities" but the viewAllHref redirects to "miscellaneous" category. These should match - both should use "student-activities" for consistency with other student activities pages like clubs.

Suggested change

	viewAllHref={`/${locale}/notifications?category=miscellaneous`}
	viewAllHref={`/${locale}/notifications?category=student-activities`}

[Copilot]

Unnecessary space manipulation in string formatting. The expression ${member.person.telephone}.trim() adds a space before the telephone and then trims it, which is redundant. Either remove the leading space or remove the trim() call. Similarly on line 274 for the year field.

Suggested change

	? ` ${member.person.telephone}`.trim()
	: '-',
	branch: branchWithDegree,
	year: year ? ` ${year}` : '-',
	? `${member.person.telephone}`.trim()
	: '-',
	branch: branchWithDegree,
	year: year ? `${year}` : '-',

[Debatreya]

@Soumil221 check this

[Copilot]

Inconsistent possessive form. The text uses "Student's Representatives" (singular possessive) but should likely be "Students' Representatives" (plural possessive) to match the context of representing multiple students. Alternatively, could be "Student Representatives" (no possessive) which is more common in institutional contexts.

Suggested change

	studentsRepresentatives: `Student's Representatives`,
	studentsRepresentatives: `Student Representatives`,