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

feat: Resume from my tickets after LOGIN #563

Merged
merged 1 commit into from Sep 20, 2018

Conversation

liveHarshit
Copy link
Member

@liveHarshit liveHarshit commented Sep 17, 2018

Fixes #497

@liveHarshit liveHarshit changed the title when redirected from my tickets to login fragment, resume from my tickets after login #497 when redirected from my tickets to login fragment, resume from my tickets after login Sep 17, 2018
@nikit19
Copy link
Member

nikit19 commented Sep 17, 2018

@liveHarshit also please add "fixes" issue number in the description so that when your PR gets merged the issue will also get closed

@iamareebjamal
Copy link
Member

Please follow commit guidelines

@liveHarshit
Copy link
Member Author

@liveHarshit also please add "fixes" issue number in the description so that when your PR gets merged the issue will also get closed

Okay, Got it. 👍

nikit19
nikit19 previously approved these changes Sep 18, 2018
Copy link
Member

@nikit19 nikit19 left a comment

Choose a reason for hiding this comment

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

We follow this guide for writing proper commit messages. Please read it https://chris.beams.io/posts/git-commit/

Copy link
Member

@simarsingh24 simarsingh24 left a comment

Choose a reason for hiding this comment

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

Rest looks good

simarsingh24
simarsingh24 previously approved these changes Sep 18, 2018
Copy link
Member

@iamareebjamal iamareebjamal left a comment

Choose a reason for hiding this comment

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

Please follow commit guidelines and squash commits

@liveHarshit
Copy link
Member Author

Updated

Copy link
Member

@nikit19 nikit19 left a comment

Choose a reason for hiding this comment

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

Please follow commit guidelines and make the commit message and the PR title same.

@liveHarshit liveHarshit changed the title when redirected from my tickets to login fragment, resume from my tickets after login feat: Resume my from my tickets from after LOGIN Sep 20, 2018
@liveHarshit liveHarshit force-pushed the development branch 3 times, most recently from 1502a17 to 2426c3c Compare September 20, 2018 18:30
@liveHarshit liveHarshit changed the title feat: Resume my from my tickets from after LOGIN feat: Resume my from my tickets after LOGIN Sep 20, 2018
@liveHarshit
Copy link
Member Author

Updated, Sorry for all the complications. I will try best by next time.

@liveHarshit liveHarshit changed the title feat: Resume my from my tickets after LOGIN feat: Resume from my tickets after LOGIN Sep 20, 2018
@iamareebjamal iamareebjamal merged commit 581489e into fossasia:development Sep 20, 2018
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.

None yet

4 participants