Skip to content

App.js file#55

Open
psankhe28 wants to merge 1 commit into
samagra-comms:monitoring-dashboardfrom
psankhe28:app
Open

App.js file#55
psankhe28 wants to merge 1 commit into
samagra-comms:monitoring-dashboardfrom
psankhe28:app

Conversation

@psankhe28
Copy link
Copy Markdown

App.js file with all the links and corresponding files.

@vercel
Copy link
Copy Markdown

vercel Bot commented Aug 28, 2023

@psankhe28 is attempting to deploy a commit to the UCI Team on Vercel.

A member of the Team first needs to authorize it.

Comment thread src/App.js
/>
<Route
path="/monitoring/logs/uci-api"
element={
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

why same component have 2 different routes ?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Monitoring is the main section and in that we have subsection. so if someone clicks on monitoring, if we do not have any route for it, it will show an err. So, just for that added the route of the first subsection

Comment thread src/App.js

return (
<div style={{ height: "100vh", overflow: "scroll", ...theme }}>
<div style={{ height: "100vh", overflow: "hidden", ...theme }}>
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

why overflow:hidden ?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Incase of scroll, we get a scrollbar both for vertical and horizontal in dashboard page

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.

2 participants