public
/
pyceo
27
1
Fork 3
Code Issues 25 Pull Requests 5 Projects Releases 1 Wiki Activity

Add positions to CEO #63

New Issue
Closed
opened 2022-06-27 00:55:17 -04:00 by r389li · 4 comments
r389li commented 2022-06-27 00:55:17 -04:00
Owner
Copy Link

Add the following positions to CEO:

  • ext affairs lead
  • marketing lead
  • design lead
  • events lead
  • reps lead
  • mods lead
  • photography lead
  • other
Add the following positions to CEO: - `ext affairs lead` - `marketing lead` - `design lead` - `events lead` - `reps lead` - `mods lead` - `photography lead` - `other`
r389li self-assigned this 2022-10-01 21:23:14 -04:00
r389li removed their assignment 2022-10-01 21:24:09 -04:00
n4chung was assigned by r389li 2022-10-01 21:33:18 -04:00
n4chung commented 2022-10-13 00:09:06 -04:00
Owner
Copy Link

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).

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).
n4chung referenced this issue from a commit 2022-10-13 18:13:01 -04:00
add new positions (#63)
merenber commented 2023-08-07 15:40:31 -04:00
Owner
Copy Link

Re-opening because the original PR got reverted (by dceb5d6572).

Re-opening because the original PR got reverted (by dceb5d6572560cc90487c5ff23b0de3ec067a615).
merenber reopened this issue 2023-08-07 15:40:31 -04:00
merenber added the
priority
very high
 label 2023-10-14 22:20:38 -04:00
n4chung pinned this 2024-01-15 22:04:02 -05:00
l227zhan commented 2024-01-22 20:26:40 -05:00
Member
Copy Link

PR #117 now passes CI testing. Ready for merge

PR #117 now passes CI testing. Ready for merge
l227zhan commented 2024-01-22 22:06:33 -05:00
Member
Copy Link

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 in tests/ceo/cli/test_positions.py. See commits in branch fix-positions for more details.

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 in ``tests/ceo/cli/test_positions.py``. See commits in branch fix-positions for more details.
merenber unpinned this 2024-02-10 11:49:31 -05:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
master
124-fix-regression
new-positions
54-newsletter
WIP-66
65-isClubRep-fix
83-sqlfstrings
60-group-lookup
63-fix-ci
1.0.22
v0.5.1
v0.4.11
v0.4.10
v0.4.9
v0.4.8
v0.4.7
v0.4.6
v0.4.5
Labels
Clear labels   
priority
high
High priority

  
priority
low
Low priority

  
priority
medium
Medium priority

  
priority
very high
Very high priority

  
BUG

   
Feature

   
High Priority

   
Low Priority

   
Medium Priority
No Label
priority
high
priority
low
priority
medium
priority
very high
BUG
Feature
High Priority
Low Priority
Medium Priority
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
n4chung
4 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#63
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?