-
Notifications
You must be signed in to change notification settings - Fork 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
Date Filter for Subscriber Home #666
Conversation
Also noticing the build errors are not displaying on the browser... going to look into that. |
} | ||
|
||
/** Custom datefilter for the subscriber home page. Control the calendar state with custom button, custom styling also applied. Also allows user to navigate a day at a time via arrow buttons. */ | ||
export const DateFilter: React.FC<IDateFilterProps> = ({ fetch }) => { |
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.
I may be blind but I don't see you using this property in the component? Remove it if you aren't.
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.
Not blind, you are correct
app/subscriber/src/features/content/list-view/interfaces/IContentListFilter.ts
Show resolved
Hide resolved
@@ -24,6 +25,7 @@ export interface IHomeFilterProps { | |||
*/ | |||
export const HomeFilters: React.FC<IHomeFilterProps> = ({ fetch }) => { |
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.
It does not appear that you are using this property in this component either. Please remove.
b0e9392
to
4e37dec
Compare
@@ -9,6 +9,7 @@ | |||
user's mobile device or desktop. See https://developers.google.com/web/fundamentals/web-app-manifest/ | |||
--> | |||
<link rel="manifest" href="%PUBLIC_URL%/manifest.json" /> | |||
<link rel="icon" type="image/x-icon" href="%PUBLIC_URL%/favicon.ico"> |
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.
Seems like manifest.json only works for local..
In this PR: