Add positions to CEO #63
Labels
No Label
priority
high
priority
low
priority
medium
priority
very high
BUG
Feature
High Priority
Low Priority
Medium Priority
No Milestone
No project
No Assignees
4 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: public/pyceo#63
Loading…
Reference in New Issue
No description provided.
Delete Branch "%!s(<nil>)"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Add the following positions to CEO:
ext affairs lead
marketing lead
design lead
events lead
reps lead
mods lead
photography lead
other
I've updated the tests config but I can't find how to initiate the tests so I haven't ran the automated tests.
I've however tested running the changes and also tested setting some roles while running CEO in development (in a Docker container).
Re-opening because the original PR got reverted (by
dceb5d6572
).PR #117 now passes CI testing. Ready for merge
PR #117 has been merged successfully. Closing issue.
For future reference:
To add new positions, add their corresponding translations to the dict in
ceo/tui/views/position_names.py
. Add positions to the corresponding .ini files in both test and ceo. Finally, update the unit test with hardcoded output intests/ceo/cli/test_positions.py
. See commits in branch fix-positions for more details.