Skip to content

Commit

Permalink
TODO list
Browse files Browse the repository at this point in the history
  • Loading branch information
CharlesTaylor7 committed May 8, 2024
1 parent 4c121e2 commit 568bf1e
Show file tree
Hide file tree
Showing 3 changed files with 20 additions and 44 deletions.
14 changes: 0 additions & 14 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -28,19 +28,6 @@ This is excessive, and I will try to stick to 3:
- debug
- info

### Installing deps

To install a new dep:
1. Comment out the 2nd line from .cargo/config.toml. So it looks like:
`# replace-with = "vendored-sources"`
2. `cargo add foo`
3. `cargo vendor`
4. Uncomment .cargo/config.toml.

You might think this is a lot of friction for adding deps. This is a good thing. Vendoring deps makes it so that the docker image can build more readily, and makes our codebase more resistant to supply chain attacks.
The friction makes me think twice before adding a new dep.


## Tech Stack
Tech Stack:
- Frontend:
Expand Down Expand Up @@ -79,7 +66,6 @@ To release:
gh run watch && ./deploy.sh
```


## Secret management

Script to generate a new signing key:
Expand Down
48 changes: 20 additions & 28 deletions TODO.md
Original file line number Diff line number Diff line change
@@ -1,19 +1,12 @@
## Defects
- [ ] etag the public/index.css or disable caching altogether.
- [ ] Prevent tracing subscriber from overwriting the env_logger

- [ ] Arrows overlap text in role logs.
- [ ] No logs for draft phase
- [ ] No logs for end of previous turn. Very easy to miss Actions of last player
- [ ] Bewitch action wipes out the whole screen and forces a page reload.

## Work
- [ ] Implement Continuous deployment.
- [ ] Game host with elevated permissions
- [ ] Auth via Supabase
- [ ] Game host with elevated permissions
- [ ] Prevent tracing subscriber from overwriting the env_logger
- [ ] Multi room support
- [ ] Optimize binary size (After removing all panics)
- Notifications
- [ ] Max height for hand panel, with horizontal scroll
- [ ] Randomize audio notification
- [ ] Notifications
- Seer: card is taken.
- Seer: card is given.
- Magician
Expand All @@ -24,25 +17,24 @@
- Theater
- Wizard

- Fixup janky UI
- [ ] The required rank 4 actions should have an indicator.
- [ ] Warlord Menu:
- Destroy cost should be displayed prominently
- should disable & grey districts that cost too much.

- [ ] Build Menu:
- Reminder that you have a warrant; emoji + text
- Cost to build should be displayed
- Use tabs instead of radio buttons

- [ ] Marshal menu should disable & grey districts that cost too much.
- [ ] Max height for hand panel, with horizontal scroll
- [ ] Randomize audio notification
## UI Defects
- [ ] Arrows overlap text in role logs.
- [ ] No logs for draft phase
- [ ] No logs for end of previous turn. Very easy to miss Actions of last player
- [ ] Bewitch action wipes out the whole screen and forces a page reload.
- [ ] The required rank 4 actions should have an indicator.
- [ ] Rank 8 Menus (Warlord, Diplomat, Marshal):
- Costs should be displayed prominently
- Should disable & grey districts that cost too much.
- [ ] Build Menu:
- Reminder that you have a warrant; emoji + text
- Cost to build should be displayed
- Use tabs instead of radio buttons

## Ideas
- [ ] Bots
- [ ] Bots to play against
- [ ] Randomizer mode: any role can be any rank
- [ ] Persist game config to sqlite db
- [ ] Persist game config
- [ ] Keyboard based warrant assignment
- [ ] Detect when building is impossible.
- [ ] Pesist the dragged position of districts in a city.
Expand Down
2 changes: 0 additions & 2 deletions deploy.sh
Original file line number Diff line number Diff line change
Expand Up @@ -11,8 +11,6 @@ supabase db push

# upload stylesheet to supabase cdn
node upload-css.js
# yes y | supabase storage --experimental rm ss:///styles/index.css || true
# supabase storage --experimental cp styles/index.min.css ss:///styles/index.css

# deploy to citadels.fly.dev
fly secrets set GIT_SHA=$(git show -s --format=%H)
Expand Down

0 comments on commit 568bf1e

Please sign in to comment.