Skip to content

issue #13 fix #2

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

Merged
merged 15 commits into from
Jun 19, 2019
Merged

Conversation

rashmi73
Copy link
Contributor

@rashmi73 rashmi73 commented Jun 9, 2019

No description provided.

@luizrrodrigues luizrrodrigues self-requested a review June 13, 2019 17:57
Copy link
Collaborator

@luizrrodrigues luizrrodrigues left a comment

Choose a reason for hiding this comment

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

@rashmi73 You need deploy build to include change in build folder too, run:

npm install
npm run build

@rashmi73
Copy link
Contributor Author

@luizrrodrigues done please check

@luizrrodrigues luizrrodrigues self-requested a review June 13, 2019 20:42
Copy link
Collaborator

@luizrrodrigues luizrrodrigues left a comment

Choose a reason for hiding this comment

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

Check feedback here: #13 (comment)

Please fix conflicts.

Thanks

luizrrodrigues pushed a commit that referenced this pull request Jun 15, 2019
luizrrodrigues pushed a commit that referenced this pull request Jun 15, 2019
@luizrrodrigues luizrrodrigues changed the title issue 2398 fix issue #13 fix Jun 15, 2019
@luizrrodrigues luizrrodrigues self-requested a review June 15, 2019 21:04
luizrrodrigues
luizrrodrigues previously approved these changes Jun 15, 2019
@luizrrodrigues luizrrodrigues dismissed their stale review June 15, 2019 21:39

Need fix conflicts

@luizrrodrigues luizrrodrigues self-requested a review June 15, 2019 21:39
Copy link
Collaborator

@luizrrodrigues luizrrodrigues left a comment

Choose a reason for hiding this comment

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

@rashmi73 Looks good now.

Just need fix conflicts.

Thanks.

@luizrrodrigues luizrrodrigues self-requested a review June 19, 2019 03:08
@luizrrodrigues
Copy link
Collaborator

@rashmi73 sorry, I merge another issue before that, can you update builds please?

Copy link
Collaborator

@luizrrodrigues luizrrodrigues left a comment

Choose a reason for hiding this comment

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

@rashmi73 Sorry, I just merge your other issue and showing conflicts again.

@luizrrodrigues luizrrodrigues self-requested a review June 19, 2019 20:26
@luizrrodrigues luizrrodrigues merged commit dd9f429 into topcoder-platform:develop Jun 19, 2019
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.

2 participants