Raymond Li r389li
Loading Heatmap…

r389li commented on pull request public/pyceo#88

Add group lookup functionality

@d6sun thanks for getting on this! Would you mind fixing the CI tests? @r345liu could you help review?

1 month ago

r389li pushed to master at public/pyceo

  • b507c56136 Show groups in member for API, CLI and TUI (#82)

2 months ago

r389li merged pull request public/pyceo#82

Show groups in member for API, CLI and TUI

2 months ago

r389li closed issue public/pyceo#69

Show groups in CEO

2 months ago

r389li commented on issue public/pyceo#60

Group/club lookup

@d6sun Yes, both TUI and CLI please. Let's actually call it `search` instead of `lookup`. If you can support fuzzy matching it'd be great, but don't worry if you can't.

3 months ago

r389li deleted branch use-admin-creds-in-http-client from public/pyceo

3 months ago

r389li pushed to master at public/pyceo

  • c0c9736593 Use the admin creds in the HTTPClient when necessary (#85)

3 months ago

r389li merged pull request public/pyceo#85

Use the admin creds in the HTTPClient when necessary

3 months ago

r389li approved public/pyceo#84

Assume program is Alumni if UWLDAP is missing data

LGTM, thanks merenber

3 months ago

r389li commented on pull request public/pyceo#82

Show groups in member for API, CLI and TUI

Code looks good, though tests need to be fixed. Great work so far!

4 months ago

r389li commented on issue public/pyceo#69

Show groups in CEO

The formatting is fine!

4 months ago

r389li commented on issue public/pyceo#69

Show groups in CEO

We can include it.

4 months ago

r389li commented on pull request public/pyceo#80

Implement TUI support for multiple users in each position

Will this clutter up log?

4 months ago

r389li suggested changes for public/pyceo#80

Implement TUI support for multiple users in each position

Almost there! Thanks for taking this on!

4 months ago

r389li commented on pull request public/pyceo#80

Implement TUI support for multiple users in each position

Would the rest of this file work as well? On a cursory glance there's a couple more places that look like they should need changing (i.e. removing old positions, etc.)

4 months ago

r389li pushed to master at public/pyceo

  • dceb5d6572 Revert "#63: Add positions to CEO (#79)"

4 months ago

r389li closed issue public/pyceo#63

Add positions to CEO

4 months ago

r389li closed issue public/pyceo#74

Sort group member listing

4 months ago