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

Support multiple Medium users #69

Open
gomorizsolt opened this issue Jan 3, 2020 · 6 comments
Open

Support multiple Medium users #69

gomorizsolt opened this issue Jan 3, 2020 · 6 comments

Comments

@gomorizsolt
Copy link
Collaborator

gomorizsolt commented Jan 3, 2020

Extracted from #68 (comment).

I can't really remember off the top of my head but we had some issues with supporting this feature. We also discussed this somewhere in the closed issues. Could anyone remember what they were exactly?

@gomorizsolt gomorizsolt added bug Something isn't working priority:backlog and removed bug Something isn't working labels Jan 3, 2020
@gomorizsolt
Copy link
Collaborator Author

Found the issue, it seems we discussed something over the phone.

https://gitlab.com/c-hive/c-hive.dev/issues/20#note_199580936

@nkapolcs
Copy link
Contributor

nkapolcs commented Jan 3, 2020

Good question! I am not sure, but I think because we want to start with MVP, that's why we made the decision to stay with one Github user, and one Medium user.

@gomorizsolt
Copy link
Collaborator Author

That makes sense, thanks. Let's leave this issue around then.

@gomorizsolt
Copy link
Collaborator Author

#111 (+ #117 => changes) makes the configuration so that it's now possible to display projects from various orgs/users. How about doing so in the case of articles? One thing that's worthwhile to keep in mind is that there's a slight difference between the URLs:

https://medium.com/feed/@zsgomori (user) vs https://medium.com/feed/c-hive (publication)

But it's rather something that's expected to be mentioned in the docs.

Proposals:

(1) Use an array of strings format.

medium:
   - "c-hive"
   - "@zsgomori"

(2) Option to set the maximum number of displayed articles per entry (therefore, the screen won't be "bloated" with dozens of articles). It'd be an optional setting.

medium:
   - org: "c-hive"
      limit: 5
   - org: "@zsgomori"

@gomorizsolt
Copy link
Collaborator Author

cc @thisismydesign

Let me know if any of the above makes sense. If so, I'll create follow-up issues and close this one.

@thisismydesign thisismydesign changed the title Clarify why multiple Medium users are not supported Support multiple Medium users Apr 28, 2020
@thisismydesign
Copy link
Member

The proposal looks good. Let's keep this in the backlog for now, we can go with proposal 1 as a first step later.

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

No branches or pull requests

3 participants