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

ZCS-8012 : Upgrade jquery in zm-admin-console #16

Merged
merged 1 commit into from
Mar 13, 2020
Merged

Conversation

auminpatel
Copy link
Contributor

@auminpatel auminpatel commented Feb 14, 2020

  • Moving jquery and clipboard code to its own modules
  • Not minifying the 3rd party code using yiucomposser instead using the minified versionthat is shipped with the 3rd party lib
  • Loading Jquery is loaded before itself has it is happening now
  • Upgrading jquery version to 3.4.1

@auminpatel auminpatel closed this Feb 19, 2020
@auminpatel auminpatel deleted the ZCS-8012 branch February 19, 2020 10:38
@auminpatel auminpatel restored the ZCS-8012 branch February 19, 2020 10:38
@auminpatel
Copy link
Contributor Author

Deleting this branch as removing the jquery from admin-console

@auminpatel auminpatel deleted the ZCS-8012 branch February 19, 2020 10:39
@auminpatel auminpatel restored the ZCS-8012 branch February 20, 2020 13:14
@auminpatel auminpatel reopened this Feb 20, 2020
@auminpatel auminpatel closed this Feb 27, 2020
@auminpatel auminpatel deleted the ZCS-8012 branch February 27, 2020 11:26
@auminpatel auminpatel restored the ZCS-8012 branch February 27, 2020 12:38
@auminpatel auminpatel reopened this Feb 27, 2020
- Moving jquery and clipboard code to its own modules
- Not minifying the 3rd party code using yiucomposser instead using the minified versionthat is shipped with the 3rd party lib
- Loading Jquery is loaded before itself has it is happening now
- Upgrading jquery version to 3.4.1
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@auminpatel auminpatel closed this Mar 13, 2020
@auminpatel auminpatel deleted the ZCS-8012 branch March 13, 2020 12:58
@auminpatel auminpatel restored the ZCS-8012 branch March 13, 2020 12:58
@auminpatel auminpatel reopened this Mar 13, 2020
@silentsakky silentsakky merged commit f48ac81 into develop Mar 13, 2020
@silentsakky silentsakky deleted the ZCS-8012 branch March 13, 2020 13:23
@jzedryx
Copy link

jzedryx commented Aug 5, 2020

How to apply this update, can you give some details step by step procedure. Im new to git i want to upgrade the jquery of my zimbra 8.8.15

@silentsakky
Copy link
Member

How to apply this update, can you give some details step by step procedure. Im new to git i want to upgrade the jquery of my zimbra 8.8.15

We are going to backport these changes to 8.8.15 patch 13, once it is released you can just apply that patch, that should be easy compared to manually applying changes

@jzedryx
Copy link

jzedryx commented Aug 7, 2020

Thanks sounds good waiting for patch 13.

@jzedryx
Copy link

jzedryx commented Aug 10, 2020

We are going to backport these changes to 8.8.15 patch 13, once it is released you can just apply that patch, that should be easy compared to manually applying changes

Hi, did you have a target date for the release of Zimbra 8.8.15 Patch 13?

Thanks

@silentsakky
Copy link
Member

We are going to backport these changes to 8.8.15 patch 13, once it is released you can just apply that patch, that should be easy compared to manually applying changes

Hi, did you have a target date for the release of Zimbra 8.8.15 Patch 13?

Thanks

It's released now https://wiki.zimbra.com/wiki/Zimbra_Releases/8.8.15/P13

@jzedryx
Copy link

jzedryx commented Aug 31, 2020

Does the update include jquery in patch13?

@silentsakky
Copy link
Member

For admin console, yes

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.

5 participants