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

Update jquery.calendars.afghan-ps.js #1283

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mohibullahiqbal111
Copy link
Contributor

Hi Fran
i just made change in names of months and days of the week but i really try to delete the English names but i cant so please delete the English names also

in the calendar we are not using short names so please use full names

thank you

Hi Fran
i just made change in names of months and days of the week but i really try to delete the English names but i cant so please delete the English names also

in the calendar we are not using short names so please use full names

thank you
@nursix
Copy link
Member

nursix commented May 7, 2016

Hi - you shouldn't change the order of the day names, this creates an inconsistency with the minified names, and accidentally changes the first day of the week.

7 maj 2016 kl. 11:52 skrev mohibullahiqbal111 [email protected]:

Hi Fran
i just made change in names of months and days of the week but i really try to delete the English names but i cant so please delete the English names also

in the calendar we are not using short names so please use full names

thank you

You can view, comment on, or merge this pull request online at:

#1283

Commit Summary

Update jquery.calendars.afghan-ps.js
File Changes

M static/scripts/calendars/i18n/jquery.calendars.afghan-ps.js (14)
Patch Links:

https://github.com/sahana/eden/pull/1283.patch
https://github.com/sahana/eden/pull/1283.diff

You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub

@mohibullahiqbal111
Copy link
Contributor Author

Hi - so i will edit again and send you new pull request and also there are some language wise problem also i will edit day by day

thank you for your kindly reply

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.

2 participants