Skip to content

Commit e16e353

Browse files
author
Rishabh-Rathod
committed
Merge branch 'release' into feature/add-s3-crud-support
2 parents 1a4fa6b + ab3fdd4 commit e16e353

File tree

89 files changed

+1968
-379
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

89 files changed

+1968
-379
lines changed

app/client/cypress/fixtures/ChartDsl.json

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -63,6 +63,24 @@
6363
"bottomRow": 6,
6464
"parentId": "0",
6565
"widgetId": "y4gcv7cjg0"
66+
},
67+
{
68+
"isVisible": true,
69+
"inputType": "TEXT",
70+
"label": "",
71+
"widgetName": "Input2",
72+
"version": 1,
73+
"resetOnSubmit": true,
74+
"type": "INPUT_WIDGET",
75+
"isLoading": false,
76+
"parentColumnSpace": 74,
77+
"parentRowSpace": 40,
78+
"leftColumn": 8,
79+
"rightColumn": 13,
80+
"topRow": 7,
81+
"bottomRow": 7,
82+
"parentId": "0",
83+
"widgetId": "y4gcv7cjg1"
6684
}
6785
]
6886
}

app/client/cypress/fixtures/application-file.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
{
2-
"appsmithVersion":"v1.5.13",
2+
"appsmithVersion":"1.5.14",
33
"exportedApplication": {
44
"userPermissions": [
55
"canComment:applications",

app/client/cypress/fixtures/testdata.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -116,6 +116,7 @@
116116
"dropdownErrorMsg": "This value does not evaluate to type \"Array<{ \"label\": \"string\", \"value\": \"string\" }>\".",
117117
"tableWidgetErrorMsg": "This value does not evaluate to type \"Array<Object>\".",
118118
"bindingDataPoint": "{{JSON.stringify(Chart1.selectedDataPoint)}}",
119+
"bindingSeriesTitle": "{{Chart1.selectedDataPoint.seriesTitle",
119120
"bindChartData": "{{Chart1.selectedDataPoint",
120121
"appUrl": "https://oauth.mocklab.io/userinfo",
121122
"clientID": "169444434892406",

app/client/cypress/integration/Smoke_TestSuite/ClientSideTests/DisplayWidgets/Chart_Data_point_spec.js

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -52,4 +52,13 @@ describe("Chart Widget Functionality", function() {
5252
});
5353
});
5454
});
55+
56+
it("Chart with seriesTitle feature validation", function() {
57+
cy.SearchEntityandOpen("Input2");
58+
cy.get(widgetsPage.defaultInput).type(testdata.bindingSeriesTitle);
59+
60+
cy.get(publish.inputWidget + " " + "input")
61+
.last()
62+
.should("have.value", dsl.dsl.children[0].chartData[0].seriesName);
63+
});
5564
});

app/client/src/actions/commentActions.ts

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -265,3 +265,13 @@ export const incrementThreadUnreadCount = () => ({
265265
export const decrementThreadUnreadCount = () => ({
266266
type: ReduxActionTypes.DECREMENT_COMMENT_THREAD_UNREAD_COUNT,
267267
});
268+
269+
export const deleteCommentThreadEvent = (thread: CommentThread) => ({
270+
type: ReduxActionTypes.DELETE_COMMENT_THREAD_EVENT,
271+
payload: thread,
272+
});
273+
274+
export const deleteCommentEvent = (comment: Comment) => ({
275+
type: ReduxActionTypes.DELETE_COMMENT_EVENT,
276+
payload: comment,
277+
});
Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
1+
import { ReduxActionTypes } from "constants/ReduxActionConstants";
2+
3+
export const setIsGitSyncModalOpen = (isOpen: boolean) => ({
4+
type: ReduxActionTypes.SET_IS_GIT_SYNC_MODAL_OPEN,
5+
payload: isOpen,
6+
});
Lines changed: 3 additions & 0 deletions
Loading
Lines changed: 3 additions & 0 deletions
Loading
Lines changed: 4 additions & 0 deletions
Loading

app/client/src/components/ads/DialogComponent.tsx

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@ const StyledDialog = styled(Dialog)<{
66
setMaxWidth?: boolean;
77
width?: string;
88
maxHeight?: string;
9+
maxWidth?: string;
910
showHeaderUnderline?: boolean;
1011
}>`
1112
&& {
@@ -15,6 +16,7 @@ const StyledDialog = styled(Dialog)<{
1516
${(props) => (props.maxHeight ? `max-height: ${props.maxHeight};` : "")}
1617
width: ${(props) => props.width || "640px"};
1718
${(props) => props.setMaxWidth && `width: 100vh;`}
19+
${(props) => props.maxWidth && `max-width: ${props.maxWidth};`}
1820
1921
& .${Classes.DIALOG_HEADER} {
2022
position: relative;
@@ -86,21 +88,24 @@ type DialogComponentProps = {
8688
width?: string;
8789
maxHeight?: string;
8890
onOpening?: () => void;
91+
onClose?: () => void;
8992
setModalClose?: (close: boolean) => void;
9093
triggerZIndex?: number;
9194
showHeaderUnderline?: boolean;
9295
getHeader?: () => ReactNode;
9396
canEscapeKeyClose?: boolean;
9497
className?: string;
98+
maxWidth?: string;
9599
};
96100

97101
export function DialogComponent(props: DialogComponentProps) {
98102
const [isOpen, setIsOpen] = useState(!!props.isOpen);
99103

100-
const { setModalClose } = props;
104+
const { onClose: onCloseProp, setModalClose } = props;
101105
const onClose = () => {
102106
setModalClose ? setModalClose(false) : null;
103107
setIsOpen(false);
108+
onCloseProp && onCloseProp();
104109
};
105110

106111
useEffect(() => {
@@ -128,6 +133,7 @@ export function DialogComponent(props: DialogComponentProps) {
128133
className={props.className}
129134
isOpen={isOpen}
130135
maxHeight={props.maxHeight}
136+
maxWidth={props.maxWidth}
131137
onClose={onClose}
132138
onOpening={props.onOpening}
133139
setMaxWidth={props.setMaxWidth}

0 commit comments

Comments
 (0)