Skip to content
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

Position Postings #572

Closed
wants to merge 8 commits into from
Closed

Position Postings #572

wants to merge 8 commits into from

Conversation

Muirrum
Copy link
Member

@Muirrum Muirrum commented Nov 4, 2021

Per #562, this creates a new django app for the position postings. The
thought process behind the position flow is:

  1. Exec posts a position
  2. Active members can see that there are open positions, and can view
    them on the DB and apply with 1 click
    (side thought on displaying -- I was thinking something similar to
    the CC report reminders but on the main page)
  3. Applicants are given a form to complete if provided by exec, and exec
    can see the applicants event/training history in the position view
  4. Exec votes and does exec-y things to pick the person, who then gets a
    congratulatory email and the position is marked as closed.

Writing this out, I need to add a field to the Position model to mark
if the position has been filled.

Per WPI-LNL#562, this creates a new django app for the position postings. The
thought process behind the position flow is:

1. Exec posts a position
2. Active members can see that there are open positions, and can view
   them on the DB and apply with 1 click
     (side thought on displaying -- I was thinking something similar to
     the CC report reminders but on the main page)
3. Applicants are given a form to complete if provided by exec, and exec
   can see the applicants event/training history in the position view
4. Exec votes and does exec-y things to pick the person, who then gets a
   congratulatory email and the position is marked as closed.

Writing this out, I need to add a field to the `Position` model to mark
if the position has been filled.
@Muirrum Muirrum changed the title positions: Create position models and add app positions: Position Postings Nov 4, 2021
@Muirrum Muirrum changed the base branch from master to dev November 4, 2021 15:45
@Muirrum Muirrum changed the title positions: Position Postings Position Postings Nov 4, 2021
@coveralls
Copy link

coveralls commented Nov 4, 2021

Coverage Status

Coverage decreased (-0.2%) to 93.872% when pulling 62f6a26 on Muirrum:position-postings into 696a1d4 on WPI-LNL:dev.

This commit will add the following templates:
- position_list
- position_detail

It will update the following models:
- Position

I'm debating just getting rid of the ApplicationInstance model entirely
and letting exec track applications through their own external tools.
Some positions won't have any reports (e.g exec board positions posted
so that potential candidates know what the job entails).
@Muirrum
Copy link
Member Author

Muirrum commented Nov 12, 2021

Closing in favor of #577

@Muirrum Muirrum closed this Nov 12, 2021
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.

2 participants