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.

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

UI Improvements #87

Merged
merged 22 commits into from
Nov 18, 2024
Merged

UI Improvements #87

merged 22 commits into from
Nov 18, 2024

Conversation

rosajsx
Copy link
Contributor

@rosajsx rosajsx commented Nov 15, 2024

This pull request includes multiple changes aimed at improving the user interface and functionality of various components in the application. The most important changes include modifications to the KeyspaceInfo and DashboardPage components, as well as updates to the ESLint configuration and Dockerfile.

Component Enhancements:

Configuration Updates:

  • .eslintrc.json: Added new ESLint rules to disable specific unicorn plugin rules for code consistency.

Dockerfile Update:

UI

Screen Recording 2024-11-15 at 14 48 00
Screen Recording 2024-11-15 at 14 38 28
Screen Recording 2024-11-15 at 14 17 19
Screen Recording 2024-11-15 at 14 16 25

@rosajsx
Copy link
Contributor Author

rosajsx commented Nov 15, 2024

closes #41

.docker/Dockerfile.bookworm Outdated Show resolved Hide resolved
src/app/(main)/page.tsx Outdated Show resolved Hide resolved
src/app/(main)/query-runner/_components/cql-editor.tsx Outdated Show resolved Hide resolved
gvieira18 and others added 5 commits November 18, 2024 10:34
### Summary

This pull request includes significant updates to the Docker and CI
configurations. The changes aim to streamline the Docker setup, improve
environment variable handling, and update the `docker-compose` network
configurations.

> [!NOTE]
> With this update, we're moving to using the Alpine image at the
development stage. So we can remove the `Dockerfile.bookworm` later.

### Changes:

- [x] Added a new `Dockerfile.alpine` with environment variables,
dependency installations, and user setup for a Node.js development
environment.
- [x] Removed the old `Dockerfile` which included multiple stages for
building and running the application.
- [x] Simplified the network configuration in `docker-compose.ghcr.yml`
and `docker-compose.yml` by removing specific IP address assignments.
- [x] Added `NEXT_TELEMETRY_DISABLED` and
`COREPACK_ENABLE_DOWNLOAD_PROMPT` environment variables in the CI
workflow configuration.

---------

Signed-off-by: Gabriel do Carmo Vieira <[email protected]>
…od-dependencies group (basementdevs#85)

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…pendencies group (basementdevs#86)

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@rosajsx rosajsx requested a review from gvieira18 November 18, 2024 13:43
Copy link
Collaborator

@gvieira18 gvieira18 left a comment

Choose a reason for hiding this comment

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

LGTM

@gvieira18 gvieira18 merged commit 6e1d2d9 into basementdevs:main Nov 18, 2024
4 checks passed
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.

2 participants