Skip to content

[4.2] Fix sampledata processed and message#37574

Merged
roland-d merged 5 commits intojoomla:4.2-devfrom
heelc29:issue-sampledata2
Jan 19, 2023
Merged

[4.2] Fix sampledata processed and message#37574
roland-d merged 5 commits intojoomla:4.2-devfrom
heelc29:issue-sampledata2

Conversation

@heelc29
Copy link
Copy Markdown
Contributor

@heelc29 heelc29 commented Apr 18, 2022

Summary of Changes

Fix attribute for sample data set already processed
Possibility to translate the message which appears onError of ajax request

Testing Instructions

  • Install a sample data set and press the same install button again (after successful execution) without reloading the page
  • Enable lang debug and create an syntax error in onAjaxSampledataApplyStep1 function of the sample data plugin

Actual result BEFORE applying this Pull Request

Request to confirm start is displayed and js error
image

No possibility to translate the message if ajax-request fails
image

Expected result AFTER applying this Pull Request

Alert if sample data set is already installed and no js error
image

Possibility to translate the message if ajax-request fails
image

Documentation Changes Required

none

@joomla-cms-bot joomla-cms-bot added Language Change This is for Translators NPM Resource Changed This Pull Request can't be tested by Patchtester PR-4.1-dev labels Apr 18, 2022
@heelc29 heelc29 changed the base branch from 4.1-dev to 4.2-dev June 25, 2022 06:37
@heelc29 heelc29 changed the title [4.1] Fix sampledata processed and message [4.2] Fix sampledata processed and message Jun 25, 2022
@Quy Quy removed the PR-4.1-dev label Jun 27, 2022
@heelc29 heelc29 force-pushed the issue-sampledata2 branch from 07f3ab8 to 831cc13 Compare July 22, 2022 19:08
@HLeithner HLeithner removed the psr12 label Oct 23, 2022
@Kostelano
Copy link
Copy Markdown
Contributor

I have tested this item ✅ successfully on 635d5e3


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

@Quy
Copy link
Copy Markdown
Contributor

Quy commented Jan 19, 2023

I have tested this item ✅ successfully on bd2528e


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

@Quy
Copy link
Copy Markdown
Contributor

Quy commented Jan 19, 2023

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Jan 19, 2023
@roland-d roland-d merged commit 725d772 into joomla:4.2-dev Jan 19, 2023
@roland-d
Copy link
Copy Markdown
Contributor

Thank you

@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Jan 19, 2023
@roland-d roland-d added this to the Joomla! 4.2.7 milestone Jan 19, 2023
@heelc29 heelc29 deleted the issue-sampledata2 branch January 19, 2023 19:20
Kostelano added a commit to JPathRu/localisation that referenced this pull request Feb 1, 2023
* Joomla 4.2.6
joomla/joomla-cms#39143 - (только для en-GB)
joomla/joomla-cms#39317 - (исправлено ранее)

* Joomla 4.2.7
joomla/joomla-cms#39376 +
joomla/joomla-cms#39336 - (только для en-GB)
joomla/joomla-cms#39629 - (только для en-GB)
joomla/joomla-cms#37574 +
joomla/joomla-cms#38781 - (только для en-GB)
joomla/joomla-cms#39677 - (исправлено ранее)
joomla/joomla-cms#39682 - (только для en-GB)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Language Change This is for Translators NPM Resource Changed This Pull Request can't be tested by Patchtester

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants