Opened 16 years ago
Closed 16 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.
Find this issue on GitHubChange History (8)
comment:1 Changed 16 years ago by
| Owner: | set to Garry Yao |
|---|---|
| Status: | new → assigned |
Changed 16 years ago by
| Attachment: | 3282.patch added |
|---|
comment:2 Changed 16 years ago by
| Keywords: | Review? added |
|---|
comment:3 Changed 16 years ago by
| Keywords: | Review- added; Review? removed |
|---|
Changed 16 years ago by
| Attachment: | 3282_2.patch added |
|---|
comment:4 Changed 16 years ago by
| Keywords: | Review? added; Review- removed |
|---|
Not sure why the original patch is broken.
comment:5 Changed 16 years ago by
| Keywords: | Review+ added; Review? removed |
|---|
Find this issue on GitHubNote: See
TracTickets for help on using
tickets.
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.