Skip to content

Remake/administration #257

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

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Remake/administration #257

wants to merge 5 commits into from

Conversation

aryansri-19
Copy link
Contributor

Administration Landing Static page
Missing staff data as db is not populated.

@heydoyouknowme0
Copy link
Contributor

heydoyouknowme0 commented Feb 3, 2025

Sab galat
redo
translations
@aryansri-19

@aryansri-19
Copy link
Contributor Author

Sab galat redo translations @aryansri-19

what galat

@heydoyouknowme0
Copy link
Contributor

image
test and fix in firefox

geekysilento
geekysilento previously approved these changes Apr 2, 2025
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 4 out of 5 changed files in this pull request and generated no comments.

Files not reviewed (1)
  • package.json: Language not supported
Comments suppressed due to low confidence (1)

app/[locale]/institute/administration/page.tsx:145

  • [nitpick] The class 'size-12' is ambiguous; consider using a more descriptive name that aligns with your design system or confirm that it is defined in your CSS.
<Icon className="size-12" />

@heydoyouknowme0
Copy link
Contributor

@aryansri-19 rebase

@aryansri-19 aryansri-19 force-pushed the remake/administration branch from f82f422 to 815e2d3 Compare April 10, 2025 19:28
@aryansri-19 aryansri-19 requested a review from Copilot April 10, 2025 19:29
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

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

Files not reviewed (1)
  • package.json: Language not supported
Comments suppressed due to low confidence (1)

app/[locale]/institute/administration/page.tsx:237

  • The pointsOfApprovalLinks array length does not match the number of items resulting from text.pointsOfApproval.slice(4), which could lead to undefined links. Consider ensuring both arrays are aligned or add a fallback to handle the absence of a corresponding link.
pointsOfApprovalLinks[index + 4] ? (

geekysilento
geekysilento previously approved these changes Apr 10, 2025
Copy link
Member

@geekysilento geekysilento left a comment

Choose a reason for hiding this comment

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

LGTM 👍

return <WorkInProgressStatus locale={locale} />;
const text = (await getTranslations(locale)).Administration;

const senateMembers = await db.query.committeeMembers.findMany({
Copy link
Contributor

Choose a reason for hiding this comment

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

add suspense

Copy link
Contributor Author

Choose a reason for hiding this comment

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

so just empty table or no table if there is no member fetched?

]}
/>

<main className="container mt-20">
Copy link
Contributor

Choose a reason for hiding this comment

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

no mains land

Copy link
Contributor Author

Choose a reason for hiding this comment

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

then?

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