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

[stable26] fix(workflowengine): Fix multiple UI issues in workflow engine admin settings #38189

Merged
merged 2 commits into from
May 16, 2023

Conversation

backportbot-nextcloud[bot]
Copy link

@backportbot-nextcloud backportbot-nextcloud bot commented May 10, 2023

  • ⚠️ This backport had conflicts that were resolved with the 'ours' merge strategy and is likely incomplete

backport of #38170

@nickvergessen nickvergessen force-pushed the backport/38170/stable26 branch from 6966026 to bac0941 Compare May 16, 2023 12:11
@nickvergessen nickvergessen requested a review from szaimen May 16, 2023 12:15
@nickvergessen nickvergessen marked this pull request as ready for review May 16, 2023 12:15
@nickvergessen nickvergessen added the 3. to review Waiting for reviews label May 16, 2023
@nickvergessen nickvergessen self-assigned this May 16, 2023
@nickvergessen nickvergessen requested review from a team, susnux and nfebe and removed request for a team May 16, 2023 12:15
Copy link
Contributor

@szaimen szaimen left a comment

Choose a reason for hiding this comment

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

🐘

@nickvergessen nickvergessen enabled auto-merge May 16, 2023 12:28
:placeholder="t('workflowengine', 'Select a filter')"
@input="updateCheck" />
<NcMultiselect v-model="currentOperator"
<NcSelect v-model="currentOperator"
:disabled="!currentOption"
:options="operators"
class="comparator"
label="name"
track-by="operator"
Copy link
Contributor

Choose a reason for hiding this comment

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

There is no track-by for NcSelect or vue-select.

Copy link
Member

Choose a reason for hiding this comment

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

Yeah, right. That's an oversight in master:
https://github.com/nextcloud/server/pull/38170/files#diff-89faeece31f41549ed11dd7b8c99eb2b830f068e75ab525526f8ed49a95688cbR7

Will send a new PR to master and backport it

Copy link
Member

Choose a reason for hiding this comment

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

Fix in #38310

@blizzz blizzz mentioned this pull request May 16, 2023
@nickvergessen nickvergessen merged commit 5ea5e7b into stable26 May 16, 2023
@nickvergessen nickvergessen deleted the backport/38170/stable26 branch May 16, 2023 16:36
@szaimen
Copy link
Contributor

szaimen commented Nov 16, 2023

/backport to stable25

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants