Skip to content
This repository has been archived by the owner on Aug 27, 2024. It is now read-only.

Adding all the basic logic for refreshing token #556

Closed
wants to merge 4 commits into from

Conversation

fokusferit
Copy link
Contributor

@fokusferit fokusferit commented Jan 8, 2019

This PR is a Work-in-Progress for the discussed issue about refreshing tokens. I think it is affecting #501 partially.

From my understanding what is necessary:

  • Provide in CheckHandler a "onRefreshTokens" function, which updates token for all Checks for given Repository

  • Provide a special API ENdpoint

  • Add Service Function in the Client

  • Add Action + Reducer (Redux State) functionality for this

  • Add a Button and connect it to the action in the RepositoryDetail Page

  • Write Tests

try{
const user = ctx.req.user;
const token = user.accessToken
const id = parseInt(ctx.params.id)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

change to repoId

@fokusferit fokusferit changed the title [WIP] adding all the basic logic for refreshing token Adding all the basic logic for refreshing token Feb 13, 2019
@fokusferit
Copy link
Contributor Author

I continue the work in #571 as this here is quite outdated.

@fokusferit fokusferit closed this Feb 26, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant