Skip to content

Correcting action logs filter due to #24992#25291

Merged
zero-24 merged 1 commit intojoomla:stagingfrom
infograf768:actionlog_filter
Jun 22, 2019
Merged

Correcting action logs filter due to #24992#25291
zero-24 merged 1 commit intojoomla:stagingfrom
infograf768:actionlog_filter

Conversation

@infograf768
Copy link
Copy Markdown
Member

Pull Request for Issue #24992 (comment)

Summary of Changes

Modifying Select User filter to display username instead of name due to the change in #24992.

Testing Instructions

After patch
Make sure the filter now proposes the username instead of name in the dropdown to be consistent with the Action

@degobbis
Copy link
Copy Markdown
Contributor

I have tested this item ✅ successfully on 0041135


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/25291.

1 similar comment
@alikon
Copy link
Copy Markdown
Contributor

alikon commented Jun 22, 2019

I have tested this item ✅ successfully on 0041135


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/25291.

@infograf768
Copy link
Copy Markdown
Member Author

See #24992 (comment)
maybe for another pr?

@ghost ghost removed the PR-staging label Jun 22, 2019
@ghost
Copy link
Copy Markdown

ghost commented Jun 22, 2019

Status "Ready To Commit".

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Jun 22, 2019
@infograf768 infograf768 added this to the Joomla 3.10.0 milestone Jun 22, 2019
@zero-24 zero-24 modified the milestones: Joomla 3.10.0, Joomla 3.9.9 Jun 22, 2019
@zero-24
Copy link
Copy Markdown
Contributor

zero-24 commented Jun 22, 2019

Merging thanks.

@zero-24 zero-24 merged commit ad07420 into joomla:staging Jun 22, 2019
@joomla-cms-bot joomla-cms-bot added PR-staging and removed RTC This Pull Request is Ready To Commit labels Jun 22, 2019
@infograf768 infograf768 deleted the actionlog_filter branch June 22, 2019 09:55
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.

5 participants