Requested updates to Pandokia, plus some more

spacetelescope / pandokia

Regression tests framework.

Other

2 stars 4 forks source link

Requested updates to Pandokia, plus some more #43

Closed cslocum closed 5 years ago

cslocum commented 5 years ago

Follow-up from #42, changes requested by Vicki.

pull the hard-coded "@stsci.edu" from the Pandokia application and move it to the glitch configuration file in etc-controller (second part is in corresponding PR in the etc-controller repo)
striped many obsolete links from the landing page; I left in and commented out a few that I know are old, but should probably stick around for a bit because I'm not totally sure what they're for.
some mild re-formatting of the landing page, mostly adding some additional space to make it less cluttered; this makes more sense now that the content is sparser.

cslocum commented 5 years ago

@vglaidler, I have a feeling you're going to want to visually see what changed. You can see the latest iteration of this PR here (on glitch), and you can see the original version on ssbwebv1 here.

Please note that if the changes on glitch are not approved and/or are contested, it's a quick switch to the master branch to revert.

cslocum commented 5 years ago

This is missing the "expected test summary" link. I'll fix that tomorrow.

cdsontag commented 5 years ago

Please note that if the changes on glitch are not approved and/or are contested, it's a quick switch to the master branch to revert.

And just so we are all clear, IIUC this sentence is true because of all the work that @cslocum put into the setup on glitch so that everything runs out of the repo (a checked out branch), and not a series of files hand-edited in (almost) random places on disk. Bravo Christine!

cslocum commented 5 years ago

The expected tests summary link is now restored.