-
Notifications
You must be signed in to change notification settings - Fork 10
Philimon/live poc #558
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
Philimon/live poc #558
Conversation
2f7693c
to
b4151a0
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Most of the way through, I'll pick up on the page object stuff tomorrow
Finished my review without any additional comments, I think these are the only things we need to really resolve, then I'd love to see a demo when it's closer to done. Thanks! |
Finished most of the reviews except one and adding the region specific changes we talked about. Also got the web server functionality running so we are good to have a demo in out meeting. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One tiny question, but it's not big enough to block landing. Good work!
Relevant Links
Bugzilla:
- 1958490
- 1955305
- 1955308
Description of Code / Doc Changes
Proof of concept for live site testing with the l10n harness
Process Changes Required
Mark the relevant boxes:
Workflow Checklist
Thank you!