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.

Sign up for GitHub

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

Jump to bottom

feat(frontend): add message history/editing to frontend #10

Merged
merged 61 commits into from
Sep 24, 2024
Merged

feat(frontend): add message history/editing to frontend #10

merged 61 commits into from
Sep 24, 2024

Conversation

RobertRosca
Copy link
Member

feat:

  • adds messages pane
  • fetch (some) bot details from mymdc automatically
  • taskfile instead of makefile
  • allow clients without bots
  • use traefik instead of nginx
  • support proxying w/ path prefix

refactor:

  • repositories generalised, made async
  • models split into their own files

@RobertRosca RobertRosca force-pushed the feat/web-interface-message-history branch from dc8f539 to 11452da Compare September 24, 2024 07:03
@RobertRosca RobertRosca changed the title Feat/web interface message history feat(frontend): add message history/editing to frontend Sep 24, 2024
Base automatically changed from feat/web-interface to main September 24, 2024 07:11
@RobertRosca RobertRosca marked this pull request as ready for review September 24, 2024 07:14
@RobertRosca
feat(frontend): add messages page
17c5e60
@RobertRosca
refactor: split models out into module
4fab6b3
@RobertRosca
feat(repositories): generalise repos, make async
7c89975
@RobertRosca
feat(mymdc): return bot id with credentials
10406ba
@RobertRosca
feat(frontend): set htmx/css path in Jinja env
e4d1c97
@RobertRosca
feat(frontend): add client/messages route
f4702e5
@RobertRosca
feat(frontend): make navbar sticky
757ea5d
@RobertRosca
feat(frontend): add button to go to messages
483c9e3
@RobertRosca
fix(build): use raw url for htmx add
74b68a5
@RobertRosca
build: add prettier/plugins to dev deps
f75136b
@RobertRosca
build: remove typer, add mail dependency, add vcr
bae3c28
@RobertRosca
fix(frontend): use function names in url_for calls
8bd1e94
@RobertRosca
feat(frontend): disable delete button
b1cc2e4 
deletion not allowed on elog zuilp
@RobertRosca
feat(client): make propagate_mode optional in update_message
76c0e71
@RobertRosca
refactor(services): raise http exceptions in services.get_client
0453764
@RobertRosca
fix(repositories): get lock when loading
def1caa
@RobertRosca
feat(repositories): reload from disk on key miss
1bddd69
@RobertRosca
feat(mymdc): fetch bot site from mymdc
807c32a
@RobertRosca
fix(services): fetch zulip bot id via zulip if missing
8bf94c5
@RobertRosca
feat(frontend): use fetch api for comment updates
e354f5c
@RobertRosca
style: apply formatting
0a0daa2
@RobertRosca
style: typing fixes
63d8d43
@RobertRosca
feat(frontend): allow for true partial loading of pages
41db021
@RobertRosca
fix(deploy): use correct env var for proxy root
d9594e8
@RobertRosca
build: rename compose files
5140a6e
@RobertRosca
build: use taskfile instead of makefile
f234533
@RobertRosca
revert: use uvicorn.run again
789fa35
@RobertRosca
feat(frontend): display message datetime instead of id
0af421e
@RobertRosca
chore(repositories): sort keys on write
4b5aac3
@RobertRosca
refactor(models): rename scoped client key to token
a9d6cda
@RobertRosca
feat(models): add base model for common fields
e7770d8
@RobertRosca
feat(repositories): store as list of objects, define index/key in model
90fb849
@RobertRosca
feat(frontend): warning sign if bot is not for main zulip elog
e94c502
@RobertRosca
test: refactor command updates
d8c5aae
@RobertRosca
build(docker): split into prod and dev layers
1152871
@RobertRosca
build(docker): use independent compose files instead of overrides
ca84327
@RobertRosca
refactor(frontend): move updateMessage function to main messages page
060c383
@RobertRosca
feat(frontend): add delete button to edit modal
059927e
@RobertRosca
chore(frontend): remove debugger statement
6d8e2ff
@RobertRosca
chore: log trusted hosts and warn if localhost on app startp
d83bed0
@RobertRosca
build(docker): update env vars
5e3a893
@RobertRosca
chore: remove unused nginx config file
4c92a1f
@RobertRosca
test(conftest): update conftest
0269e85
@RobertRosca
test(models): update model tests
10af37a
@RobertRosca
feat(services): use new repository style
449cbb9
@RobertRosca
feat(frontend): generalise alert to info/warning
8449b3e
@RobertRosca
feat(frontend): display warning if no messages retrieved
22f52fe
@RobertRosca
feat(frontend): show created at and created by columns in list
a5948bc
@RobertRosca
feat(repositories): return None on no entry instead of KeyError
62906e8
@RobertRosca
feat(models): move defaults out of base models
d74dd75
@RobertRosca
feat(repositories): move sorting out of repository class
5757cf1
@RobertRosca
feat(services): split out create bot to function
2edb89d
@RobertRosca
feat(mymdc): return full response dict from get zulip bot credentials
dc42c4d
@RobertRosca RobertRosca force-pushed the feat/web-interface-message-history branch from 11452da to dc42c4d Compare September 24, 2024 07:15
@RobertRosca RobertRosca merged commit ad49060 into main Sep 24, 2024
1 of 3 checks passed
@RobertRosca RobertRosca deleted the feat/web-interface-message-history branch September 24, 2024 07:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@RobertRosca