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

[staging] Fix jalali calendar javascript #28700

Merged
merged 1 commit into from
Aug 6, 2021

Conversation

infograf768
Copy link
Member

#28698

Summary of Changes

Adding missing return

Testing Instructions

Switch to Persian in administrator and edit/create an article Look at the calendar fields in the Publishing tab.

I personally have not remarked any differences in the calendar but code makes sense

@balvard
Would be interested in learning what is modified in the calendar display after the patch.

@Quy
Copy link
Contributor

Quy commented Jul 26, 2021

I have tested this item ✅ successfully on 3f6edf0

Code review


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

1 similar comment
@richard67
Copy link
Member

I have tested this item ✅ successfully on 3f6edf0

Code review


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

@richard67
Copy link
Member

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Jul 26, 2021
@zero-24 zero-24 added this to the Joomla 3.10.0 milestone Aug 6, 2021
@zero-24
Copy link
Contributor

zero-24 commented Aug 6, 2021

Merging thanks :)

@zero-24 zero-24 merged commit f9b55e9 into joomla:staging Aug 6, 2021
@joomla-cms-bot joomla-cms-bot added Language Change This is for Translators and removed RTC This Pull Request is Ready To Commit labels Aug 6, 2021
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