Skip to content

lalith #1

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

Draft
wants to merge 25 commits into
base: main
Choose a base branch
from
Draft

lalith #1

wants to merge 25 commits into from

Conversation

lalithbseervi
Copy link
Collaborator

@lalithbseervi lalithbseervi commented Jul 10, 2025

build: migrate from commonJS to ES modules:

  • migrated from CommonJS to ES modules as it is easier to work with

refactor: modularize code and improve readability:

  • separated router logic into middleware and services to improve readability.
  • added JS docs for certain functions
  • corrected the emailRegex as it wasn't correctly handling illegal email formats
  • wrote wrapper functions (refer this) to avoid duplicating code

fix (schema.sql): syntactical errors and circular foreign key references:

  • fix circular foreign key reference between events and teams table
  • fix syntactical error (missing closing bracket for checking length of lead_name in teams table

fix: add missing NOT NULL checks:

  • had forgotten to add NOT NULL checks for certain fields, added them in
  • remove the length constraint on qr_string as it is yet to be determined

fix: db now initialises automatically:

  • file restructuring performed yesterday broke the automatic db initialisation. this was fixed and the code for the same now lies solely in backend/db/utils/execute.js

feat: add generic update record function

  • added a generic function to execute the UPDATE commands using the id attribute as the identifier

feat: create and edit events

  • added routing and logic for creating and editing events.
  • added validation for validating event dates and names.

refactor: use common table for all non-participants and add roles for the same

  • added the shortlisted bool field in the participant table
  • used a common table dbAuthorisedUsers that will hold records of all non-participants (i.e, admins, organisers, volunteers)
  • addded a role field to differentiate between admins, organisers, and volunteers

feat: add generic select function

  • wrote the generic select function to query the database
  • it is designed to support combinations of 'AND' and 'OR'

feat: auto-check whether an admin exists

  • queries the db for admins. if no admin exists, then one is created using details from the console input.

feat: integrate google oauth

  • Integrated the Google OAuth component into frontend.
  • Wrote handlers on the backend to verify the credentialResponse.credentials JWT token received from frontend
  • If the token is valid, two http-only cookies namely 'access' and 'refresh' are set with appropriate expiry

fix: semantic errors in checkAdminRecords and promptAdminCreation

  • Added missing await and one missing export in the automatic admin creation flow

@lalithbseervi lalithbseervi changed the title Lalith lalith: PR #1 Jul 11, 2025
@lalithbseervi lalithbseervi changed the title lalith: PR #1 lalith Jul 11, 2025
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.

1 participant
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy