Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Feature: add "outdated" notification on download and submit #802

Open
SleeplessByte opened this issue Feb 1, 2019 · 6 comments
Open

Feature: add "outdated" notification on download and submit #802

SleeplessByte opened this issue Feb 1, 2019 · 6 comments

Comments

@SleeplessByte
Copy link
Member

If a student downloads an exercise or submits an exercise that has a newer version of test suite, add the message that their version is outdated and they could update to the latest version. Preferably via a flag in the CLI.

Slack source in #questions

@Smarticles101
Copy link
Member

How does the api currently handle this? I imagine it always downloads the latest version of the problem. @iHiD ?

@SleeplessByte
Copy link
Member Author

@Smarticles101 yep it always downloads the newest unless you've downloaded an older version before. Then when you submit, it always just works and you must press the update button on the site and download again in order to update.

@iHiD
Copy link
Member

iHiD commented Feb 5, 2019

We could even make it interactive :)

@cmccandless
Copy link

Then when you submit, it always just works and you must press the update button on the site and download again in order to update.

I just had a use case where apparently I had downloaded the exercise before, never submitted and deleted the exercise directory (TBH, I have no idea why I cleared the directory, thus removing any local evidence of a previous download 🤷‍♂️). When I redownloaded, the test suite was out-of-date, but the update button does not appear on the site until a submission has been made.

It would be nice if that button were visible without submission.

@iHiD
Copy link
Member

iHiD commented Feb 12, 2019

@cmccandless Mind opening an issue in exercism/exercism for this please, as I think it's a seperate issue and I don't want it to get lost in this thread.

@cmccandless
Copy link

@iHiD Done. See exercism/exercism#4694

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants