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(condo): DOMA-9269 move receipts upload to extension tab #5027

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

abshnko
Copy link
Member

@abshnko abshnko commented Jul 29, 2024

[WIP] New flow for uploading receipts in integrations
Now uploading will be in the extension tab with history of uploading

@abshnko abshnko added the 🔬 WIP Not intended to be merged right now, it is a work in progress label Jul 29, 2024
@@ -59,7 +59,7 @@ export const MainContent: React.FC<MainContentProps> = ({
result.push({
label: extensionPageTitle,
key: EXTENSION_TAB_KEY,
children: <IFrame src={appUrl} reloadScope='organization' withPrefetch withLoader withResize/>,
children: <IFrame src={appUrl} reloadScope='organization' withPrefetch withLoader withResize scrolling='yes' />,
Copy link
Member

Choose a reason for hiding this comment

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

it's better to control it by child window. @SavelevMatthew check it please

onLoad={handleLoad}
hidden={isLoading || isError || hidden}
ref={handleRefChange}
height={frameHeight}
// NOTE: Deprecated, but overflow: hidden still not works in Chrome :)
scrolling='no'
scrolling={scrolling}
Copy link
Member

Choose a reason for hiding this comment

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

@abshnko abshnko force-pushed the feat/condo/DOMA-9269/new-registry-upload-form branch from d49040b to bfcdc4f Compare August 2, 2024 06:16
@abshnko abshnko force-pushed the feat/condo/DOMA-9269/new-registry-upload-form branch from bfcdc4f to 89ab71e Compare August 2, 2024 06:20
@abshnko abshnko force-pushed the feat/condo/DOMA-9269/new-registry-upload-form branch from 89ab71e to 8bc0513 Compare August 2, 2024 08:35
@abshnko abshnko force-pushed the feat/condo/DOMA-9269/new-registry-upload-form branch from 8bc0513 to 1a4f735 Compare August 2, 2024 14:21
@abshnko abshnko force-pushed the feat/condo/DOMA-9269/new-registry-upload-form branch from 1a4f735 to 3e26acf Compare August 5, 2024 07:40
@abshnko abshnko force-pushed the feat/condo/DOMA-9269/new-registry-upload-form branch from 3e26acf to 7546c0f Compare August 19, 2024 14:45
@abshnko abshnko force-pushed the feat/condo/DOMA-9269/new-registry-upload-form branch from 00b22f0 to 6e6a7e7 Compare August 20, 2024 09:24
@abshnko abshnko force-pushed the feat/condo/DOMA-9269/new-registry-upload-form branch from 6e6a7e7 to 8809f56 Compare August 22, 2024 05:44
@abshnko abshnko force-pushed the feat/condo/DOMA-9269/new-registry-upload-form branch from a9d0194 to 59944da Compare August 28, 2024 11:26
Copy link

sonarcloud bot commented Aug 28, 2024

@abshnko abshnko force-pushed the feat/condo/DOMA-9269/new-registry-upload-form branch from 59944da to 672b52a Compare October 2, 2024 10:56
@abshnko abshnko force-pushed the feat/condo/DOMA-9269/new-registry-upload-form branch from 42ab681 to 05c0430 Compare October 7, 2024 06:52
@abshnko abshnko force-pushed the feat/condo/DOMA-9269/new-registry-upload-form branch from 5b28164 to cbe3c60 Compare October 7, 2024 09:59
@abshnko abshnko force-pushed the feat/condo/DOMA-9269/new-registry-upload-form branch from cbe3c60 to 3f4ed45 Compare October 7, 2024 11:01
@abshnko abshnko force-pushed the feat/condo/DOMA-9269/new-registry-upload-form branch from 3f4ed45 to a20c364 Compare October 7, 2024 14:12
@abshnko abshnko force-pushed the feat/condo/DOMA-9269/new-registry-upload-form branch from 3e6f94f to 82efdd2 Compare October 10, 2024 00:46
Copy link

sonarcloud bot commented Oct 10, 2024

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔬 WIP Not intended to be merged right now, it is a work in progress
Development

Successfully merging this pull request may close these issues.

2 participants