Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

rewrite the goggles login system to use id.wmo #529

Closed
1 of 3 tasks
hannahkane opened this issue Aug 5, 2015 · 13 comments
Closed
1 of 3 tasks

rewrite the goggles login system to use id.wmo #529

hannahkane opened this issue Aug 5, 2015 · 13 comments
Labels

Comments

@hannahkane
Copy link
Collaborator

We need to:

  • update the banner on published makes (currently reads that Goggles are part of Mozilla's Webmaker toolset)
  • add a link to the original page that was remixed, as per Legal's request
  • determine other polishing work needed

/cc @Pomax - any flags on the above?

@Pomax
Copy link

Pomax commented Aug 6, 2015

Practically speaking, we cannot update goggles makes. In part because of how we intended the goggles to be used from the get go, but also due to technical choices made along the way: the notice is not a banner that we generate on top of goggle pages, it's literally hardcoded into every goggles make, so in order to update it, we would have to run through the database of goggle makes, parse each record, find the notice (in its various historical incarnations, which is tricky) and then update it to contain a new text. If we also want to include a link to the original site (which is an odd thing to ask for now that we're shutting it down, after four years of that never having been an issue) then things get quite a bit harder still, so as an assessment of the effort required to do this, my conclusion would have to be that, realistically, we cannot do this.

@hannahkane
Copy link
Collaborator Author

Thanks for the info, @Pomax.

One correction - we're not shutting Goggles down; we're moving it to mozilla.org where it will live under the Teach site.

In theory (meaning: "with a bunch of work"), could we add this information to Goggles makes going forward, even if we're not able to apply this retroactively to existing makes?

/cc @simonwex @ldecoursy

@Pomax
Copy link

Pomax commented Aug 6, 2015

The goggles codebase got too complicated to maintain, so any changes to how we want it to work (short of simply running it off a new domain) basically means we need to take out a few weeks to create a new library with the same capabilities as the goggles, but written as a maintainable library instead, and switch over to that instead. The goggles code as it exists now has effectively been in "we don't touch it" mode for a while now, pending the decision and free time to scrap the current implementation and redo it.

@hannahkane hannahkane added P2 and removed P1 labels Aug 14, 2015
@hannahkane
Copy link
Collaborator Author

cc @mmmavis see ^

@hannahkane
Copy link
Collaborator Author

Moving to next heartbeat

@hannahkane
Copy link
Collaborator Author

Related tickets are filed here: https://github.com/mozilla/teach.webmaker.org/labels/The%20Goggles

@hannahkane
Copy link
Collaborator Author

This ticket no longer reflects the work to be done. Goggles-related work for this heartbeat is mozilla/learning.mozilla.org#1220, so I'm updating this ticket's description

@hannahkane hannahkane changed the title Updates to Goggles makes rewrite the goggles login system to use id.wmo Sep 9, 2015
@OpenMatt
Copy link

@hannahkane status? [flagged!]

@hannahkane
Copy link
Collaborator Author

will check in with @Pomax today. not sure of status.

@Pomax
Copy link

Pomax commented Sep 11, 2015

using id.wmo works, although the UX for it is rather poor at the moment and will need some polish. Login for goggles now works similarly to how we deal with discourse.wmo, where you get a new window/tab for logging into webmaker, which you can close once login finishes.

@hannahkane
Copy link
Collaborator Author

thanks @Pomax. Pushing to next heartbeat for polish.

@OpenMatt
Copy link

@hannahkane status? [flagged!]

@hannahkane
Copy link
Collaborator Author

This ticket is a dupe of #554

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

No branches or pull requests

3 participants