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

Add Telescope Developer login account and blog for local dev #3501

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

humphd
Copy link
Contributor

@humphd humphd commented Apr 16, 2022

We need some data to use locally with the parser service, database, login flow, etc. This adds a few things:

  1. A new user that you can login to the login service with. We need to follow this up with an easy way on startup to add this user to the db
  2. A "blog" at http://localhost:8888/telescope-developer-blog/index.html with a feed at http://localhost:8888/telescope-developer-blog/feed.xml that we can use to register with the parser.

We need to figure out how to add this content to the database on startup, and also how to deal with the http://localhost issue in docker (e.g., parser won't be able to download a blog from localhost:8888).

@gitpod-io
Copy link

gitpod-io bot commented Apr 16, 2022

@humphd humphd mentioned this pull request Apr 17, 2022
8 tasks
@aserputov aserputov self-requested a review April 18, 2022 19:43
TueeNguyen
TueeNguyen previously approved these changes Apr 18, 2022
Copy link
Contributor

@TueeNguyen TueeNguyen left a comment

Choose a reason for hiding this comment

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

Not sure about the saml php file but the rest looks ok

@humphd
Copy link
Contributor Author

humphd commented Apr 20, 2022

I don't know that this is needed, if we can automate the steps @TueeNguyen is suggesting for testing the parser service.

@DukeManh, @TueeNguyen what do you think?

DukeManh
DukeManh previously approved these changes Apr 20, 2022
Copy link
Contributor

@DukeManh DukeManh left a comment

Choose a reason for hiding this comment

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

I think we should have this example account for local dev while figuring out how to automate the steps.

@manekenpix manekenpix force-pushed the master branch 12 times, most recently from 94eec21 to e27efa5 Compare December 3, 2022 21:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants