Skip to content
This repository was archived by the owner on Mar 13, 2025. It is now read-only.

fix: issue #124 #129

Merged
merged 2 commits into from
Feb 26, 2021
Merged

fix: issue #124 #129

merged 2 commits into from
Feb 26, 2021

Conversation

mbaghel
Copy link
Contributor

@mbaghel mbaghel commented Feb 26, 2021

No description provided.

Replaced email logic for extension request with Email Popup
Copy link
Contributor

@maxceem maxceem left a comment

Choose a reason for hiding this comment

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

@mbaghel generally works great and the code is well-structured!

There are few small code improvements I'd like to make. Please, kindly check the comments below.

@mbaghel mbaghel requested a review from maxceem February 26, 2021 13:34
Copy link
Contributor

@maxceem maxceem left a comment

Choose a reason for hiding this comment

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

Thanks, @mbaghel all is perfect now.

@maxceem maxceem merged commit 295aecb into topcoder-archive:dev Feb 26, 2021
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.

2 participants