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

[GEN-1096] refactor: product rebranding #1389

Closed

Conversation

alonkeyval
Copy link
Collaborator

No description provided.

Copy link

alonkeyval and others added 29 commits November 3, 2024 15:22
This pull request includes several updates to the
`frontend/graph/schema.resolvers.go` file, focusing on adding
functionality and improving existing resolver methods.

### Functional Improvements:

* Added the `time` package to handle time formatting and conversions.
* Implemented the `Type` resolver method for the `Destination` model to
return the type as a string instead of panicking with an error.
* Implemented the `Conditions` resolver method for the `Destination`
model, converting `LastTransitionTime` to a string pointer and
populating the conditions list with the converted data.
This pull request includes several changes to the `frontend/webapp`
directory, focusing on refactoring the theme provider, adding a new
header component, and updating the layout file to use the new theme
provider. The most important changes are summarized below:

### Refactoring and Theming:

* Replaced the `ThemeProvider` with `ThemeProviderWrapper` in
`frontend/webapp/app/layout.tsx` to encapsulate the theme and
styled-components registry.
[[1]](diffhunk://#diff-788a38ec94bfca9467b13cb6eb93cdeccd4237facfb5fff310e05b3af67413efL5-R6)
[[2]](diffhunk://#diff-788a38ec94bfca9467b13cb6eb93cdeccd4237facfb5fff310e05b3af67413efL17-R26)
* Removed the old `palette.ts` file and moved theme-related code to
`theme-provider/index.tsx`.
[[1]](diffhunk://#diff-80a3cfb572b6f3e0b18eb5880735fe84c8cdbc222e7af63c3a696c732221aeeaL1-L50)
[[2]](diffhunk://#diff-1b2f072fc24d4e371e711036a6ec0d8437c8b6b68c97378c19fa865cff06e0a1R1-R15)
* Added `StyledComponentsRegistry` for server-side rendering of
styled-components.

### Component Additions:

* Added a new `SetupHeader` component in
`frontend/webapp/components/setup/header/index.tsx` for the setup page
header.
* Updated the `index.tsx` file in `frontend/webapp/components/setup` to
export the new `SetupHeader` component.

### File Organization:

* Updated `frontend/webapp/styles/index.ts` to export the new
`theme-provider` module instead of the old `palette`.
Co-authored-by: Tamir David <[email protected]>
Co-authored-by: Tamir David <[email protected]>
Co-authored-by: Amir Blum <[email protected]>
Co-authored-by: Ben Elferink <[email protected]>
Co-authored-by: Ron Federman <[email protected]>
Co-authored-by: yodigos <[email protected]>
Co-authored-by: Mike Dame <[email protected]>
Co-authored-by: Ben Elferink <[email protected]>
Co-authored-by: Tamir David <[email protected]>
Co-authored-by: Tamir David <[email protected]>
Co-authored-by: Amir Blum <[email protected]>
Co-authored-by: Ron Federman <[email protected]>
Co-authored-by: yodigos <[email protected]>
Co-authored-by: Mike Dame <[email protected]>
Co-authored-by: alonbraymok <[email protected]>
Co-authored-by: Ben Elferink <[email protected]>
Co-authored-by: Tamir David <[email protected]>
Co-authored-by: Tamir David <[email protected]>
Co-authored-by: Amir Blum <[email protected]>
Co-authored-by: Ron Federman <[email protected]>
Co-authored-by: yodigos <[email protected]>
Co-authored-by: Mike Dame <[email protected]>
Co-authored-by: alonbraymok <[email protected]>
Co-authored-by: Ben Elferink <[email protected]>
Co-authored-by: Tamir David <[email protected]>
Co-authored-by: Tamir David <[email protected]>
Co-authored-by: Amir Blum <[email protected]>
Co-authored-by: Ron Federman <[email protected]>
Co-authored-by: yodigos <[email protected]>
Co-authored-by: Mike Dame <[email protected]>
Co-authored-by: alonbraymok <[email protected]>
@blumamir blumamir closed this Nov 20, 2024
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.

4 participants