ComputerScienceHouse / ComputerScienceHouse/packet

Show badges for Alumni and Advisors

Open
#138 0 comments 0 reactions 0 assignees View on GitHub
ux enhancement
Dominant language
Python
Stars
25
Forks
33
PR merge metrics
No merged PRs in 30d

Description

Right now we don't show badges for alumni and advisors due to the nature of it requiring an ldap sync.

1. Call LDAP on first misc signatures of a packet to collect roles
2. On subsequent signatures, check to see if they've signed another packet, if so, pull roles from previous signature to avoid ldap call
3. Update `ldap-sync` to update roles on all transitions

(We may also want to move in the schema a `roles` table so that we only pull the data once per person overall)

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by tracing the packet signature handling and the `ldap-sync` entry point mentioned in the issue. Confirm how roles are currently obtained and persisted, then verify that first signatures query LDAP, later signatures reuse prior roles, and role transitions update all affected records; any schema change should be evaluated separately.

Written by the indexing model from the issue text.

Assessment

Tech stack
flask, python, sqlalchemy
Domain
authorization, backend, database
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.