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

[API] Activity API #5666

Closed
mmarif4u opened this issue Jan 8, 2019 · 9 comments · Fixed by #23494
Closed

[API] Activity API #5666

mmarif4u opened this issue Jan 8, 2019 · 9 comments · Fixed by #23494
Labels
issue/confirmed Issue has been reviewed and confirmed to be present or accepted to be implemented modifies/api This PR adds API routes or modifies them type/proposal The new feature has not been accepted yet but needs to be discussed first.
Milestone

Comments

@mmarif4u
Copy link

mmarif4u commented Jan 8, 2019

Hi,

Is there an API for activity/dashboard view?

I also noticed that there is private variable used in api/v1/user/repos API for private/public access type. In java it is reserved keyword and cannot be used. Is there a way around it?. I need it for showing lock icon.

Thanks

@lunny lunny added the type/question Issue needs no code to be fixed, only a description on how to fix it yourself. label Jan 9, 2019
@mmarif4u
Copy link
Author

For the private variable part, I will use serialization.

Hope to get any updates on Activity API as that is going to be the home screen of the app so that user can see all the activities. :)

@stale
Copy link

stale bot commented Mar 17, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs during the next 2 weeks. Thank you for your contributions.

@stale stale bot added the issue/stale label Mar 17, 2019
@lunny lunny added issue/confirmed Issue has been reviewed and confirmed to be present or accepted to be implemented and removed issue/stale labels Mar 17, 2019
@mmarif4u mmarif4u changed the title [API] Activity API and private var in user/repo API [API] Activity API Oct 8, 2019
@lunny lunny added type/proposal The new feature has not been accepted yet but needs to be discussed first. modifies/api This PR adds API routes or modifies them and removed type/question Issue needs no code to be fixed, only a description on how to fix it yourself. labels Mar 29, 2023
@lunny
Copy link
Member

lunny commented Mar 29, 2023

@mmarif4u Looks like this issue is for a long time. Have you resolve the problem yourself? I'm sorry I have no android phone so I cannot test the great APP.

@mmarif4u
Copy link
Author

@lunny unfortunately no, we are still waiting for an API to utilize it.

@Zettat123
Copy link
Contributor

I create a PR(#23494) to add activity API. But it has not been merged :)

@lunny
Copy link
Member

lunny commented Mar 29, 2023

And think it more, since it's a restful API, the android has to check it every internal. It will waste much electricity. Maybe a websocket implementation or a blocked HTTP request is better.

@mmarif4u
Copy link
Author

Well that depends, we don't call an API unless you are in the active view that way we reduce the calls. User has the option to refresh it through pull.

@mmarif4u
Copy link
Author

@Zettat123 thanks for the work on this.

@lunny lunny added this to the 1.20.0 milestone Mar 29, 2023
@lunny
Copy link
Member

lunny commented Mar 29, 2023

Well that depends, we don't call an API unless you are in the active view that way we reduce the calls. User has the option to refresh it through pull.

OK. So I think #23494 may fix this issue.

lunny pushed a commit that referenced this issue Apr 4, 2023
Close #5666

Add APIs for getting activity feeds.
@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 20, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
issue/confirmed Issue has been reviewed and confirmed to be present or accepted to be implemented modifies/api This PR adds API routes or modifies them type/proposal The new feature has not been accepted yet but needs to be discussed first.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants