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

NEW Wait for promises before submitting forms #1792

Conversation

emteknetnz
Copy link
Member

@emteknetnz emteknetnz commented Jun 24, 2024

@emteknetnz emteknetnz force-pushed the pulls/2/always-inline-submit-elements branch from 21713fe to 16702b3 Compare June 26, 2024 02:35
@emteknetnz emteknetnz force-pushed the pulls/2/always-inline-submit-elements branch from 16702b3 to e5e01ae Compare July 4, 2024 06:43
@emteknetnz emteknetnz marked this pull request as ready for review July 4, 2024 07:00
@emteknetnz
Copy link
Member Author

Broken --prefer-lowest build is unrelated and will be fixed in silverstripe/gha-ci#132

client/src/legacy/LeftAndMain.js Outdated Show resolved Hide resolved
@emteknetnz emteknetnz force-pushed the pulls/2/always-inline-submit-elements branch from e5e01ae to e789c40 Compare July 11, 2024 01:32
Copy link
Member

@GuySartorelli GuySartorelli left a comment

Choose a reason for hiding this comment

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

LGTM, will merge when elemental PR is approved

@emteknetnz emteknetnz force-pushed the pulls/2/always-inline-submit-elements branch from e789c40 to 2a14985 Compare July 18, 2024 06:58
Copy link
Member

@GuySartorelli GuySartorelli left a comment

Choose a reason for hiding this comment

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

LGTM, will merge when elemental PR is approved

@emteknetnz emteknetnz force-pushed the pulls/2/always-inline-submit-elements branch from 2a14985 to be057e5 Compare July 18, 2024 23:40
@emteknetnz emteknetnz force-pushed the pulls/2/always-inline-submit-elements branch from be057e5 to 4140199 Compare July 19, 2024 00:03
Copy link
Member

@GuySartorelli GuySartorelli left a comment

Choose a reason for hiding this comment

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

LGTM, will merge when elemental PR is approved

@emteknetnz emteknetnz force-pushed the pulls/2/always-inline-submit-elements branch from 4140199 to 7e4d882 Compare July 22, 2024 02:35
@GuySartorelli
Copy link
Member

CI failure will be resolved as part of silverstripe/.github#285

@GuySartorelli
Copy link
Member

Will rerun CI after #1792 is merged

@GuySartorelli
Copy link
Member

Can you please rebase this? Looks like it needs a rebase to pick up the missing @job tags in behat.

@emteknetnz emteknetnz force-pushed the pulls/2/always-inline-submit-elements branch from 7e4d882 to ef32740 Compare July 23, 2024 03:59
@emteknetnz
Copy link
Member Author

Done

Copy link
Member

@GuySartorelli GuySartorelli left a comment

Choose a reason for hiding this comment

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

LGTM

@GuySartorelli GuySartorelli merged commit 7e3f3a5 into silverstripe:2 Jul 24, 2024
16 checks passed
@GuySartorelli GuySartorelli deleted the pulls/2/always-inline-submit-elements branch July 24, 2024 21:33
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