-
Notifications
You must be signed in to change notification settings - Fork 0
Feature/command pattern #17
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
base: develop
Are you sure you want to change the base?
Conversation
b7a4206 to
9134b2e
Compare
| PrivateQuery: () => null, | ||
| PublicQuery: () => null, | ||
| AppEnumCollection: () => null, | ||
| DailyStandUpType: () => null, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This might be incorrect.
|
@thenav56 We need to update the type for bulk create/update of entries before we can merge this. There's error because of type mismatch on graphql. |
- Implement saving changes in the server (in background) - Hide undo and redo if action is not available - Remove unused functions and files - Remove usage of confirm button on delete
9134b2e to
df0f385
Compare
- move "saving" icon on App - show "saving" icon
- use red underline instead of red background
- lighten saturday and sunday - increase left sidebar
- decrease size of timur icon on header - remove un-necessary links - make navbar actions more verbose - remove "jump to date" from navbar - use icons on undo/redo buttons
- change all search behavior with fuzzy search
| children: React.ReactNode; | ||
| } | ||
|
|
||
| function AuthProvider(props: BaseProps) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's separate all these context functions into different files
Depends on
Changes