Skip to content

[4.3] Guided Tours search by note#40607

Merged
obuisard merged 5 commits intojoomla:4.3-devfrom
brianteeman:guidednote
Jun 19, 2023
Merged

[4.3] Guided Tours search by note#40607
obuisard merged 5 commits intojoomla:4.3-devfrom
brianteeman:guidednote

Conversation

@brianteeman
Copy link
Contributor

Summary of Changes

All components that have a note field allow you to search by the value of the note. This was missed in com_guidedtours

Testing Instructions

Add a string into the note field for a tour
Search for that string in the list view

Repeat for steps

Actual result BEFORE applying this Pull Request

No matching results

Expected result AFTER applying this Pull Request

Matching results

image

This is not a new feature. It is a bug that it was not implemented

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

All components that have a note field allow you to search by the value of the note. This was missed in com_guidedtours
@Quy
Copy link
Contributor

Quy commented May 16, 2023

Please add note to filter search description.

40607-note

@joomla-cms-bot joomla-cms-bot added the Language Change This is for Translators label May 16, 2023
@richard67 richard67 added Updates Requested Indicates that this pull request needs an update from the author and should not be tested. bug and removed Updates Requested Indicates that this pull request needs an update from the author and should not be tested. labels May 21, 2023
@Quy
Copy link
Contributor

Quy commented Jun 2, 2023

I have tested this item ✅ successfully on 17bf4bf


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

1 similar comment
@viocassel
Copy link
Contributor

I have tested this item ✅ successfully on 17bf4bf


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

@richard67 richard67 removed Language Change This is for Translators bug labels Jun 11, 2023
@richard67
Copy link
Member

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Jun 11, 2023
@richard67 richard67 added Language Change This is for Translators bug labels Jun 11, 2023
@obuisard obuisard added this to the Joomla! 4.3.3 milestone Jun 19, 2023
@obuisard obuisard merged commit e0f8e29 into joomla:4.3-dev Jun 19, 2023
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Jun 19, 2023
@obuisard
Copy link
Contributor

Thank you Brian @brianteeman for the missing note search

@brianteeman brianteeman deleted the guidednote branch June 20, 2023 02:32
Kostelano added a commit to JPathRu/localisation that referenced this pull request Aug 20, 2023
Новое в Joomla 4.3.3 Stable
* joomla/joomla-cms#40637 +
* joomla/joomla-cms#40698 - (исправлено ранее)
* joomla/joomla-cms#40716 - (только для en-GB)
* joomla/joomla-cms#40732 +
* joomla/joomla-cms#40607 +

Новое в Joomla 4.3.4 Stable
* joomla/joomla-cms#40449 +
* joomla/joomla-cms#40119 +
* joomla/joomla-cms#40297 +
* "Вход на сайт", но "Выход из системы" - разброс формулировок исправлен на термин "система", подразумевая именно вход/авторизацию пользователя, а не просто серфинг страницы сайта без авторизации
* Другие мелкие исправления
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Language Change This is for Translators

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants