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

feat(web): vite poc #6009

Draft
wants to merge 16 commits into
base: next
Choose a base branch
from
Draft

feat(web): vite poc #6009

wants to merge 16 commits into from

Conversation

scopsy
Copy link
Contributor

@scopsy scopsy commented Jul 8, 2024

TODO:

  • Light Dark Mode not picking up proper styles
  • Fix fonts so that original 14px fonts are converted to 0.875rem instead of 1rem

@@ -2,7 +2,7 @@
<html lang="en">
<head>
<!-- Google Tag Manager -->
<% if ( process.env.REACT_APP_NOVU_GTM_ID ) { %>
<!-- <% if ( process.env.REACT_APP_NOVU_GTM_ID ) { %>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Need to find a replacement

const handleOnChange = useDebouncedCallback(async (type: OnChangeType, data: any, id?: string) => {
onChange(type, data, id);
}, TYPING_DEBOUNCE_TIME_MS);
*/

const handleOnChange = (type: OnChangeType, data: any, id?: string) => {};
Copy link
Contributor Author

Choose a reason for hiding this comment

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

TODO: Fix

Comment on lines +1 to 2
export const useDebouncedCallback = function () {}; /*
export { useDebouncedCallback } from '@mantine/hooks';
Copy link
Contributor Author

Choose a reason for hiding this comment

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

TODO: This is failing the build for some reason

@@ -39,7 +39,7 @@ REDIS_CLUSTER_KEY_PREFIX=
SYNC_PATH=http://127.0.0.1:3001
API_ROOT_URL=http://127.0.0.1:3000
DISABLE_USER_REGISTRATION=false
PORT=1337
PORT=1336
Copy link
Contributor

Choose a reason for hiding this comment

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

Aligning on port 1336 for api testing.

@scopsy scopsy changed the title feat: vite poc feat(web): vite poc Aug 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants