search

ncarchedi / swirl_OLD

We've moved! See README below for more info.
http://swirlstats.com
47 stars 28 forks source link

Use menu for module question when signing in as admin? #18

Closed WilCrofter closed 10 years ago

WilCrofter commented 10 years ago

When you sign in as boring admin, swirl asks for course, module, and line number. The module question must be spelled out exactly, Module1, Module2, etc. Anything else will produce an error and abort the run. A menu of choices would be handier.

ncarchedi commented 10 years ago

I agree and this has occurred to me in the past - just never got around to fixing it. Feel free to add if it would make debugging easier for you. Correct me if I'm wrong, but we should be able to imitate the code that we already have that detects courses automatically and populates the opening menu of course selections.

Also, I'm sorry I haven't merged your other code yet. I haven't had time to look at it closely. I promise to do so today or tomorrow.

Nick

On Tue, Oct 8, 2013 at 9:22 AM, William R Bauer notifications@github.comwrote:

When you sign in as boring admin, swirl asks for course, module, and line number. The module question must be spelled out exactly, Module1, Module2, etc. Anything else will produce an error and abort the run. A menu of choices would be handier.

— Reply to this email directly or view it on GitHubhttps://github.com/ncarchedi/swirl/issues/18 .

Nicholas A. Carchedi ScM Student, Biostatistics Johns Hopkins Bloomberg School of Public Health 301.802.0573 | nick.carchedi@gmail.com

WilCrofter commented 10 years ago 
Right about reuse. I'll branch and
  modify, then submit a pull request. though not before tomorrow,
  probably.
  No hurry on whitespace branch. It's a hack anyway, but allows us
  (wife, Gina, and I) to get through all modules.
  On 10/8/2013 9:34 AM, Nick Carchedi wrote:
I agree and this has occurred to me in the past - just
  never got around to

  fixing it. Feel free to add if it would make debugging easier for
  you.

  Correct me if I'm wrong, but we should be able to imitate the code
  that we

  already have that detects courses automatically and populates the
  opening

  menu of course selections.

  Also, I'm sorry I haven't merged your other code yet. I haven't
  had time to

  look at it closely. I promise to do so today or tomorrow.

  Nick

  On Tue, Oct 8, 2013 at 9:22 AM, William R Bauer
  <notifications@github.com>wrote:

  > When you sign in as boring admin, swirl asks for course,
  module, and line

  > number. The module question must be spelled out exactly,
  Module1, Module2,

  > etc. Anything else will produce an error and abort the run. A
  menu of

  > choices would be handier.

  >

  > —

  > Reply to this email directly or view it on
  GitHub<https://github.com/ncarchedi/swirl/issues/18>

  > .

  >

  -- 
  Nicholas A. Carchedi

  ScM Student, Biostatistics

  Johns Hopkins Bloomberg School of Public Health

  301.802.0573 | nick.carchedi@gmail.com
  —
    Reply to this email directly or view
      it on GitHub.