• Joined on 2023-06-23
scott opened issue TWS/meta#18 2023-08-26 13:55:10 +00:00
Docker internal user concerns
scott commented on pull request TWS/kalkutago#15 2023-08-26 13:19:18 +00:00
Feature: User auth

That makes sense. Though I think I might do /user/{name} if we add public profile pages (and /me for the logged in user)

scott commented on pull request TWS/kalkutago#15 2023-08-26 13:17:19 +00:00
Feature: User auth

Ah, you raise a good point there. It was originally intended to, but then I changed it so that only the events update the state. Thanks for that.

scott commented on pull request TWS/kalkutago#15 2023-08-26 12:48:19 +00:00
Feature: User auth

It seems to support a UUID column type, but I don't see documentation for it, or for using it as a primary key. Is that important?

scott commented on pull request TWS/kalkutago#15 2023-08-26 12:45:00 +00:00
Feature: User auth

From the Vue docs:

This also means any component importing store can mutate it however…

scott commented on pull request TWS/kalkutago#15 2023-08-26 12:40:18 +00:00
Feature: User auth

These are generated by the sea-orm CLI 🤷

scott commented on pull request TWS/kalkutago#15 2023-08-26 12:39:39 +00:00
Feature: User auth

It's been a while since I looked into it, but I think I remember reading something in their docs which suggested that they might be overkill for this situation.

scott pushed to feature/user-auth at scott/kalkutago 2023-08-26 11:03:28 +00:00
dafdd491f9 Move logOut function into method on state
scott pushed to feature/user-auth at scott/kalkutago 2023-08-26 10:54:51 +00:00
003383e455 Fixed logout flow
5aa16762f7 Quick tweak
db3ef7640a Renamed a variable
Compare 3 commits »
scott pushed to feature/user-auth at scott/kalkutago 2023-08-26 10:18:09 +00:00
f32a188750 Merge pull request 'Frontend was merged prematurely, this fixes that' (#3) from frontend/feature/login-view into feature/user-auth
bfffacabf6 Add logout button
290218eefe Restore logged-in state on page reload
ffc1c6806a Fix shell.nix
37426aaa52 WIP: Add busy indicators to login view
Compare 6 commits »
scott merged pull request scott/kalkutago#3 2023-08-26 10:18:07 +00:00
Frontend was merged prematurely, this fixes that
scott pushed to frontend/feature/login-view at scott/kalkutago 2023-08-26 10:17:49 +00:00
bfffacabf6 Add logout button
290218eefe Restore logged-in state on page reload
ffc1c6806a Fix shell.nix
37426aaa52 WIP: Add busy indicators to login view
db72a6df17 Add busy-indicator to login/signup buttons
Compare 6 commits »
scott created pull request scott/kalkutago#3 2023-08-26 10:16:30 +00:00
Frontend was merged prematurely, this fixes that
scott pushed to feature/user-auth at scott/kalkutago 2023-08-26 10:15:16 +00:00
3dbe2d2327 Merge pull request 'Backend feature: user auth' (#1) from backend/feature/user-auth into feature/user-auth
f6fb736ff7 Add log-out route (necessary to delete secret cookie)
a8e4e5145b add tests
1c400e7ffa Remove no-longer-existing Rust feature default_free_fn
d7285a84bb Fix authenticated track insertion
Compare 12 commits »
scott merged pull request scott/kalkutago#1 2023-08-26 10:15:14 +00:00
Backend feature: user auth
scott pushed to feature/user-auth at scott/kalkutago 2023-08-26 10:12:11 +00:00
scott pushed to frontend/feature/login-view at scott/kalkutago 2023-08-26 10:09:40 +00:00
168289335b Add logout button
dd2c2e7021 Restore logged-in state on page reload
a451fe411a Fix shell.nix
2917a96b4f WIP: Add busy indicators to login view
5f83f17a01 Add busy-indicator to login/signup buttons
Compare 16 commits »
scott pushed to backend/feature/user-auth at scott/kalkutago 2023-08-26 10:09:15 +00:00
f6fb736ff7 Add log-out route (necessary to delete secret cookie)
a8e4e5145b add tests
1c400e7ffa Remove no-longer-existing Rust feature default_free_fn
d7285a84bb Fix authenticated track insertion
205a3b165e Rename user -> users
Compare 11 commits »
scott commented on pull request TWS/meta#15 2023-08-24 15:41:27 +00:00
Expanded on details

Oh, one thing, I think a lot of this might be simplified by creating a template repo rather than relying too much on documentation. E.G. putting placeholder README, LICENSE, .gitignore, etc.

scott commented on pull request TWS/meta#15 2023-08-24 13:15:40 +00:00
Expanded on details

There could be another section that states how to fork from the CLI

As far as I'm aware this is not possible. The issue I had isn't with using the web UI for the fork button, it was the idea…