lakesare / memcode

Spaced-repetition: with real formatting.
http://memcode.com
MIT License
339 stars 73 forks source link

Regression: I no longer have any button that takes me to edit the course on desktop #187

Closed iopq closed 1 year ago

iopq commented 1 year ago

image

Expected: I should have a link to https://www.memcode.com/courses/7296/ Result: I only have two links to https://www.memcode.com/courses/7296/review so I can't edit the course without manually editing the URL

it used to be that pinning the course gave me the link I wanted

lakesare commented 1 year ago

You can access the course editing page by clicking here.

image
iopq commented 1 year ago

Thanks, that works, but it requires two clicks from the front page. Any idea to make it more convenient?

lakesare commented 1 year ago

Uh I think you're right, it's a regression, we had both the #view and #review/#learn access previously (I changed it in order to have 18 hours-before-next-review label for pinned courses, too).
I'll make it so that previous behaviour is restored but we also get the 18 hours-before-next-review label & hit you up, thanks for drawing my attention to this.

iopq commented 1 year ago

Thanks, it's resolved for me

lakesare commented 1 year ago

It's not, I didn't push anything yet, you just used some other device that has the old version of a site cached.

iopq commented 1 year ago

Haha, you're right. Have you considered cache busting when you deploy? I used to do it with file names, but maybe there's an easier way

lakesare commented 1 year ago

I'd really like to have cache busting, in fact it causes outages for people when I change the backend and their frontend cache is yet to refresh. We use service workers for caching however, so it's not super obvious how to do it (I spent a few hours on this).

lakesare commented 1 year ago

@iopq, now this is fixed!