|  | Commit message (Collapse) | Author | Age | Lines | 
|---|
| ... |  | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | | | - Now it will only show the amount of days.
- Quality of Life: Also show `day` instead of `days` when it's just 1 day. | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | | | - Update docstrings for `build_defcon_msg()` and `send_defcon_log()` - It is now taking in an `Action` directly instead of a string. | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | | | - Remove the unneccessary `get_info()` method of Enum `Action`.
- Pass Enum `Action` directly to `build_defcon_msg()` and `send_defcon_log()` - Right now, only `_defcon_action()` is using them. | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | | | - Renamed `DefconInfoLog` to `Action`
- Added `ActionInfo` namedtuple.
- Use `Action` consistently for `_defcon_action()` and `send_defcon_log()` | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | | | - Further unify defcon actions in `_defcon_action` - Thanks to Mark's suggestions.
- Changed from a Dict to an Enum for defcon log info. | 
| | | | | | |_|_|_|_|_|_|/  
| | | | |/| | | | | | |   
| | | | | | | | | | | |   
| | | | | | | | | | | |   
| | | | | | | | | | | |   
| | | | | | | | | | | |   
| | | | | | | | | | | |   
| | | | | | | | | | | |   
| | | | | | | | | | | |   
| | | | | | | | | | | |   
| | | | | | | | | | | |   
| | | | | | | | | | | |   
| | | | | | | | | | | |   
| | | | | | | | | | | |   
| | | | | | | | | | | |   
| | | | | | | | | | | |   
| | | | | | | | | | | |   
| | | | | | | | | | | |   
| | | | | | | | | | | |   
| | | | | | | | | | | |   
| | | | | | | | | | | |   
| | | | | | | | | | | |   
| | | | | | | | | | | |   
| | | | | | | | | | | |   
| | | | | | | | | | | |   
| | | | | | | | | | | |   
| | | | | | | | | | | |   
| | | | | | | | | | | |   
| | | | | | | | | | | |   
| | | | | | | | | | | |   
| | | | | | | | | | | |   
| | | | | | | | | | | |   
| | | | | | | | | | | |   
| | | | | | | | | | | |   
| | | | | | | | | | | |   
| | | | | | | | | | | |   
| | | | | | | | | | | |   
| | | | | | | | | | | |   
| | | | | | | | | | | |   
| | | | | | | | | | | |   
| | | | | | | | | | | |   
| | | | | | | | | | | | | #### Closes #539
This pull request fixes the issue for wrong status in the embed, as well as unifying certains if else to prevent similar future errors.
Most notable code is here:
```python
log_msg = f"**Staffer:** {actor} (`{actor.id}`)\n"
if change.lower() == "enabled":
    icon = Icons.defcon_enabled
    color = Colours.soft_green
    status_msg = "DEFCON enabled"
    log_msg += f"**Days:** {self.days.days}\n\n"
elif change.lower() == "disabled":
    icon = Icons.defcon_disabled
    color = Colours.soft_red
    status_msg = "DEFCON enabled"
elif change.lower() == "updated":
    icon = Icons.defcon_updated
    color = Colour.blurple()
    status_msg = "DEFCON updated"
    log_msg += f"**Days:** {self.days.days}\n\n"
```
To remedy this issue, the class will now have a private class variable
```py
_defcon_log_info: Dict[str, Tuple] = {
    'enabled': (Icons.defcon_enabled, Colours.soft_green, "**Days:** {days}\n\n"),
    'disabled': (Icons.defcon_disabled, Colours.soft_red, ""),
    'updated': (Icons.defcon_updated, Colour.blurple(), "**Days:** {days}\n\n"),
}
```
Another big change is the introduction of `_defcon_action` to unify the process all defcon actions has to go through:
- Try to do the action.
- Log error if there is any.
- Send log message. | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | | - Update test cases for in_channel_check | 
| | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | | - Change doc string for cog_check method | 
| | | | | | | | | | | | | |  | 
| | | | | | |/ / / / / /  
| | | | |/| | | | | | |  | 
| | | | | | | | | | | | 
| | | | | | | | | | | 
| | | | | | | | | | | 
| | | | | | | | | | | 
| | | | | | | | | | | | Avoid aving `Your reminder will arrive in 59 minutes and 59 seconds!`
instead of `Your reminder will arrive in 1 hour!` | 
| | | | | | | | | | | | 
| | | | | | | | | | | 
| | | | | | | | | | | 
| | | | | | | | | | | | It is now truly backward compatible and use a more DRY method | 
| | | | | | | |\ \ \ \ \  
| |_|_|_|_|_|/ / / / /  
|/| | | | | | | | | | |  | 
| |\ \ \ \ \ \ \ \ \ \ \  
| |_|_|_|/ / / / / / /  
|/| | | | | | | | | | | Fix rule alias. | 
| | |\ \ \ \ \ \ \ \ \ \  
| |/ / / / / / / / / /  
|/| | | | | | | | | | |  | 
| |\ \ \ \ \ \ \ \ \ \ \  
| |_|_|_|/ / / / / / /  
|/| | | | | | | | | | | Prevent too similar off-topic channel names | 
| | |\ \ \ \ \ \ \ \ \ \  
| |/ / / / / / / / / /  
|/| | | | | | | | | | |  | 
| | |\ \ \ \ \ \ \ \ \ \  
| | | |_|_|/ / / / / /  
| | |/| | | | | | | | |  | 
| | | | | | | | | | | | |  | 
| | | | | | | | | | | | |  | 
| | | | | | | | | | | | |  | 
| | |_|/ / / / / / / /  
|/| | | | | | | | |   
| | | | | | | | | |   
| | | | | | | | | |   
| | | | | | | | | |   
| | | | | | | | | | | Allow rule alias to take rule numbers, passes them to the `site rules`
command. Rules are now 1-indexed to conform with the representation on
the website. | 
| | | | | | | | | | | |  | 
| | | | | | | | | | | |  | 
| | | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | | | Co-Authored-By: Sebastiaan Zeeff <[email protected]> | 
| | | | | | |\ \ \ \ \ |  | 
| | | | | | | |\ \ \ \ \ |  | 
| | | | | | |/ / / / / / |  | 
| | | | | | | | | | | | 
| | | | | | | | | | | 
| | | | | | | | | | | 
| | | | | | | | | | | 
| | | | | | | | | | | 
| | | | | | | | | | | 
| | | | | | | | | | | 
| | | | | | | | | | | | Fix some liting error
Correct error
Fix linting (maybe) | 
| | | | | | |\ \ \ \ \ \ |  | 
| | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | | - More informative docstrings for `Reddit.channel` and `get_top_posts`
- Add the `amount` parameter, defaulting to 5.
- Pin a max of 12 weeks worth of top posts. | 
| | |_|_|_|_|_|_|/ / / /  
|/| | | | | | | | | | |  | 
| |\ \ \ \ \ \ \ \ \ \ \  
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | | Create the !mention command.
Co-authored-by: null <[email protected]> | 
| | |\ \ \ \ \ \ \ \ \ \ \  
| |/ / / / / / / / / / /  
|/| | | | | | | | | | | |  | 
| |\ \ \ \ \ \ \ \ \ \ \ \  
| | | | | | | | | | | | | 
| | | | | | | | | | | | | | Add raw command | 
| | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | | | Closes #334 | 
| |\ \ \ \ \ \ \ \ \ \ \ \ \  
| |_|_|_|/ / / / / / / / /  
|/| | | | | | | | | | | | | Change pep command to use `.txt` by default | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | | | Switch around trying order (txt first, then rst) | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | | | to fix the write history of the file | 
| | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | | | - Lock the role if the message wait has timed out
- Sleep, lock role and send notification if mention by staff member found.
- Lock role and send notification if mention by non-staff member found. | 
| | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | |  |