Skip to content

Add Metrological to Developer - #2273

Merged
Carlgo11 merged 6 commits into
2factorauth:masterfrom
JasperV:master
Jan 11, 2017
Merged

Carlgo11 merged 6 commits into
2factorauth:masterfrom
JasperV:master

Conversation

@JasperV

@JasperV JasperV commented Dec 21, 2016

Copy link
Copy Markdown
Contributor
  • update TransIP with more details

@Jawshy Jawshy added the add site Issue/PR adds a site to the repo. label Dec 21, 2016

@Jawshy Jawshy left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks again for submitting another pull request, @JasperV.

Please use the site's landing page - not their login page - for the url.

Does the site have any public documentation for how they have implemented two-factor authentication?

@JasperV

JasperV commented Dec 21, 2016

Copy link
Copy Markdown
Contributor Author

Hi, @Jawshy
I have updated the site url as requested.

No, they do not have any documentation on this.

@Carlgo11

Carlgo11 commented Jan 5, 2017

Copy link
Copy Markdown
Member

@JasperV can you provide us with screenshots of the process then? (With personal information blurred out)

@Carlgo11

Copy link
Copy Markdown
Member

Hello again @JasperV,
could you please provide us with the screenshots as requested earlier?
If I don't get any response within a couple of days I'll have to close this PR due to inactivity.

Thanks!
//Carl

@JasperV

JasperV commented Jan 11, 2017

Copy link
Copy Markdown
Contributor Author

Hi @Carlgo11, sorry for my delayed response. I will do my best to add the screenshots this week. Where should I add them?

@Carlgo11

Copy link
Copy Markdown
Member

Great!
You can add them using GitHub's native attachment posting.
screenshot

@JasperV

JasperV commented Jan 11, 2017

Copy link
Copy Markdown
Contributor Author

image
@Carlgo11, here is the screenshot. When you toggle the Two-step authentication, a QR code is generated in place. There is not anymore to it. Hope this is acceptable.

A screenshot was used as documentation as the site appears to lack public documentation.

@Jawshy Jawshy left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, @JasperV.

@Carlgo11
Carlgo11 merged commit b1d45c9 into 2factorauth:master Jan 11, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

add site Issue/PR adds a site to the repo.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants