Opened 17 years ago
Closed 17 years ago
#3282 closed Bug (fixed)
Skin files are not being loaded with timestamp
| Reported by: | Frederico Caldeira Knabben | Owned by: | Garry Yao |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.0 |
| Component: | General | Version: | |
| Keywords: | Confirmed Review+ | Cc: |
Description
Looking at file requests in nightly build (skins) sample, it's possible to note that the skin.js file and the main CSS files don't have the timestamp ("t") parameter in the URL. We must ensure this parameter is properly sent to the server.
Attachments (2)
Change History (8)
comment:1 Changed 17 years ago by
| Owner: | set to Garry Yao |
|---|---|
| Status: | new → assigned |
Changed 17 years ago by
| Attachment: | 3282.patch added |
|---|
comment:2 Changed 17 years ago by
| Keywords: | Review? added |
|---|
comment:3 Changed 17 years ago by
| Keywords: | Review- added; Review? removed |
|---|
Changed 17 years ago by
| Attachment: | 3282_2.patch added |
|---|
comment:4 Changed 17 years ago by
| Keywords: | Review? added; Review- removed |
|---|
Not sure why the original patch is broken.
comment:5 Changed 17 years ago by
| Keywords: | Review+ added; Review? removed |
|---|

The patch is broken. I'm not able to apply it.
Even though, the changes look by reading the patch file directly. But, I would prefer applying it for a final check.