-
Notifications
You must be signed in to change notification settings - Fork 31
Added Loading Modal to Details pages #221
Added Loading Modal to Details pages #221
Conversation
* permissions and limitations under the License. | ||
*/ | ||
|
||
export { GenerateReportLoadingModal } from './loading_modal'; |
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.
why need this, you already export this from loading_modal.tsx
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.
So when we export it in other files we don't need to write import from '../../loading_modal/loading_modal
, eliminates need for nested import
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 think loading_modal is like a util components, which may better be placed under utils
for other components to call. Otherwise, if you write a index.ts for every small component, the project will have too many files. Just my 2 cents, may refer to some project structure standard later when we do the refactor
Issue #, if available:
N/A
Description of changes:
Extract
GenerateReportLoadingModal
and added it toReport details
andReport definition details
for file format download.By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.