Update available positions in configuration files, pertaining to issue #63 #117

Merged
l227zhan merged 6 commits from new-positions into master 2024-01-22 22:00:32 -05:00
Member

Working on adding new positions and deleting legacy positions from CEO.

Testing CI unit testing

Working on adding new positions and deleting legacy positions from CEO. Testing CI unit testing
l227zhan added 1 commit 2024-01-21 22:08:30 -05:00
continuous-integration/drone/pr Build is failing Details
067a559b5a
Update available positions in configuration files, pertaining to issue #63
Added:
`Ext Affairs Lead`
`Marketing Lead`
`Design Lead`
`Events Lead`
`Reps Lead`
`Mods Lead`
`Photography Lead`
`Codey Bot Lead`
`Other`

Removed:
`IMAPD`
`Librarian`

as roles have been vacant.

Updated relevant tests
Author
Member

Also I realize that I named my branch contrary to convention, but I don't have permissions to rename branches

Also I realize that I named my branch contrary to convention, but I don't have permissions to rename branches
l227zhan added 1 commit 2024-01-22 19:38:49 -05:00
continuous-integration/drone/pr Build is failing Details
c9ad827c8b
I need new test data, added spaces to cli output
l227zhan added 2 commits 2024-01-22 19:43:40 -05:00
l227zhan added 1 commit 2024-01-22 19:47:39 -05:00
continuous-integration/drone/pr Build is failing Details
bd453990c9
Merge branch 'master' into new-positions
l227zhan added 1 commit 2024-01-22 20:08:55 -05:00
Author
Member

This change now passes unit tests, as well as a newly added unit test.

This change now passes unit tests, as well as a newly added unit test.
l227zhan changed title from WIP: Update available positions in configuration files, pertaining to issue #63 to Update available positions in configuration files, pertaining to issue #63 2024-01-22 20:25:35 -05:00
l227zhan requested review from n4chung 2024-01-22 20:25:57 -05:00
merenber approved these changes 2024-01-22 20:45:09 -05:00
merenber left a comment
Owner

Looks good to me. Thank you for taking on this long-standing issue!

Looks good to me. Thank you for taking on this long-standing issue!
n4chung approved these changes 2024-01-22 22:00:12 -05:00
n4chung left a comment
Owner

Incredible work Leon!

Incredible work Leon!
l227zhan merged commit 25994af312 into master 2024-01-22 22:00:32 -05:00
Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: public/pyceo#117
No description provided.