-
Notifications
You must be signed in to change notification settings - Fork 1
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
Cleanup/dont use eisbuk in app #783
Merged
Merged
Commits on Jul 9, 2023
-
Configuration menu - View commit details
-
Copy full SHA for 60cf185 - Browse repository at this point
Copy the full SHA 60cf185View commit details -
Create 'LayoutController' component:
* Create the componenent in 'client' package (as a single source of truth for the app layout * Use the controller on each page (passing in only the page-specific content) instead of the 'Layout' from the UI
Configuration menu - View commit details
-
Copy full SHA for 7b336bd - Browse repository at this point
Copy the full SHA 7b336bdView commit details -
* Make the Layout component in 'ui' package accept 'logo' as JSX.Elem…
…ent (instead of a component) * Use EisbukAvatar in stories, but not in production
Configuration menu - View commit details
-
Copy full SHA for ff575e3 - Browse repository at this point
Copy the full SHA ff575e3View commit details -
* Update 'createOrganization' function to accept (optional) 'displayN…
…ame' and add it to the organization * Update '/debug' page to create organization with default display name ("EisBuk Dev")
Configuration menu - View commit details
-
Copy full SHA for af75a32 - Browse repository at this point
Copy the full SHA af75a32View commit details -
Configuration menu - View commit details
-
Copy full SHA for 24b08d6 - Browse repository at this point
Copy the full SHA 24b08d6View commit details -
Extract LayoutContent as a separate component:
* LayoutContent is scrollable 'main' element * This allows us to render the CalendarNav outside the scrollable container (i.e. keep it fixed as we scroll) * LayoutContent also applies 'content-container' by default (use 'wide=true' to override) * Apply said updates to all pages in the client app
Configuration menu - View commit details
-
Copy full SHA for 747732a - Browse repository at this point
Copy the full SHA 747732aView commit details -
Configuration menu - View commit details
-
Copy full SHA for 0653983 - Browse repository at this point
Copy the full SHA 0653983View commit details -
Configuration menu - View commit details
-
Copy full SHA for 7ce5076 - Browse repository at this point
Copy the full SHA 7ce5076View commit details
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.