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

Correcting Jalali/Persian calendar popup #17432

Merged
merged 1 commit into from
Aug 8, 2017

Conversation

infograf768
Copy link
Member

@infograf768 infograf768 commented Aug 7, 2017

Pull Request for Issue #17389

Summary of Changes

Days incorrectly displayed in popup

Testing Instructions

Please test on clean staging : https://github.com/joomla/joomla-cms/archive/staging.zip
Patch proposed by @balvard
After patch we should get

screen shot 2017-08-07 at 09 05 10

@balvard @farhadst
You are the only ones to know what should be. 😄

@balvard
Copy link

balvard commented Aug 7, 2017

Salam
I checked staging file.
and I saw no change on it.
screen shot 2017-08-07 at 12 43 16
Patch Links file is right.javascript:
https://patch-diff.githubusercontent.com/raw/joomla/joomla-cms/pull/17432.patch.
I can't approve current staging file.


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

@dgrammatiko
Copy link
Contributor

@balvard your code is in this PR, not in staging. So you have to apply this PR to get the updated code

@balvard
Copy link

balvard commented Aug 7, 2017

Good
I approved it.

@balvard
Copy link

balvard commented Aug 7, 2017

Yes
Test OK with new javascript edition.
Thank you.


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

@ghost
Copy link

ghost commented Aug 7, 2017

@balvard i altered your Test as successfully at Issue Tracker.

@farhadst
Copy link

farhadst commented Aug 7, 2017

I have tested this item ✅ successfully on d5c7f25
and patch ok
I approved it.


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

@joomla-cms-bot joomla-cms-bot removed Language Change This is for Translators PR-staging labels Aug 7, 2017
@ghost
Copy link

ghost commented Aug 7, 2017

RTC after two successful tests.

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Aug 7, 2017
@infograf768 infograf768 added this to the Joomla 3.8.0 milestone Aug 8, 2017
@mbabker mbabker merged commit 2d7f8dd into joomla:staging Aug 8, 2017
@joomla-cms-bot joomla-cms-bot added Language Change This is for Translators PR-staging and removed RTC This Pull Request is Ready To Commit labels Aug 8, 2017
@infograf768 infograf768 deleted the persian_calendar branch August 9, 2017 08:02
izharaazmi added a commit to izharaazmi/joomla-cms that referenced this pull request Aug 9, 2017
* staging: (148 commits)
  Correcting non-escaped double quotes in en-GB.plg_sampledata_testing.ini (joomla#17455)
  Correct namespace reference (Fix joomla#17448)
  Correcting Jalali/Persian calendar popup (joomla#17432)
  Adding russian calendar language file (joomla#17443)
  Reset for dev
  Prepare 3.8 Beta release
  Fix covers tags
  Fix file paths
  Move library files to just libraries/src as it should be (joomla#17441)
  Add a default empty array for the session queue (joomla#16943)
  [3.8] Restructure version constants (joomla#16169)
  Adjusting copyright and versions and two remaining "sampledata" (joomla#17435)
  PHP 7.2 has branched, update Travis config to reflect
  PHP 7.2 count warning (joomla#16840)
  Enforce array for subform values (joomla#16733)
  System URL menu link (joomla#17419)
  Don't use array merge here. (joomla#17391)
  add the checked attribute (joomla#17336)
  [RFC] Mod sample data (joomla#7680)
  Rename Page to Menu Item (joomla#17409)
  ...
izharaazmi added a commit to izharaazmi/joomla-cms that referenced this pull request Aug 9, 2017
* staging: (148 commits)
  Correcting non-escaped double quotes in en-GB.plg_sampledata_testing.ini (joomla#17455)
  Correct namespace reference (Fix joomla#17448)
  Correcting Jalali/Persian calendar popup (joomla#17432)
  Adding russian calendar language file (joomla#17443)
  Reset for dev
  Prepare 3.8 Beta release
  Fix covers tags
  Fix file paths
  Move library files to just libraries/src as it should be (joomla#17441)
  Add a default empty array for the session queue (joomla#16943)
  [3.8] Restructure version constants (joomla#16169)
  Adjusting copyright and versions and two remaining "sampledata" (joomla#17435)
  PHP 7.2 has branched, update Travis config to reflect
  PHP 7.2 count warning (joomla#16840)
  Enforce array for subform values (joomla#16733)
  System URL menu link (joomla#17419)
  Don't use array merge here. (joomla#17391)
  add the checked attribute (joomla#17336)
  [RFC] Mod sample data (joomla#7680)
  Rename Page to Menu Item (joomla#17409)
  ...
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants