Skip to content

Add initial UI#58

Draft
s0nea wants to merge 2 commits intoclyso:mainfrom
s0nea:wip-basic-ui
Draft

Add initial UI#58
s0nea wants to merge 2 commits intoclyso:mainfrom
s0nea:wip-basic-ui

Conversation

@s0nea
Copy link

@s0nea s0nea commented Mar 19, 2026

The PR adds the initial UI setup, including the login page and a builds overview table.

Signed-off-by: Tatjana Dehler tatjana.dehler@clyso.com

Add a optional redirect URL.

Signed-off-by: Tatjana Dehler <tatjana.dehler@clyso.com>
@s0nea s0nea force-pushed the wip-basic-ui branch 3 times, most recently from 957daf0 to ab94954 Compare March 20, 2026 15:34
@s0nea s0nea marked this pull request as ready for review March 23, 2026 08:41
@s0nea
Copy link
Author

s0nea commented Mar 23, 2026

@jecluis @UweSchwaeke can you please take a look at the PR?
Especially at auth.py as well as the nginx configuration and the proxy container definition.

Do we want the UI to be available all the time or only if configured?

Thank you! 🙂

Copy link
Collaborator

@UweSchwaeke UweSchwaeke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jecluis @UweSchwaeke can you please take a look at the PR?
Especially at auth.py as well as the nginx configuration and the proxy container definition.

Do we want the UI to be available all the time or only if configured?

Thank you! 🙂

i will look into it

@s0nea s0nea marked this pull request as draft March 24, 2026 14:26
@jecluis
Copy link
Contributor

jecluis commented Mar 25, 2026

@s0nea isn't there a package.json or something missing?

Signed-off-by: Tatjana Dehler <tatjana.dehler@clyso.com>
@s0nea
Copy link
Author

s0nea commented Mar 26, 2026

@s0nea isn't there a package.json or something missing?

@jecluis oh, yes, indeed, it's missing. Thank you! I had to remove package.json and yarn.lock from the .gitignore file. Is it okay with you?

Does it still make sense to update the PR? I set it back to draft because of the backend part.

@jecluis
Copy link
Contributor

jecluis commented Mar 26, 2026

@s0nea please push the package.json please, so I can figure out how to test the container deployment (functionality is not relevant right now, but I really want to make sure the dev and prod container builds work with cbsd-rs)

@s0nea
Copy link
Author

s0nea commented Mar 26, 2026

@s0nea please push the package.json please, so I can figure out how to test the container deployment (functionality is not relevant right now, but I really want to make sure the dev and prod container builds work with cbsd-rs)

@jecluis pushed it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants