This Pull Request is regarding the issue #81.
Metadata Update from @jflory7: - Pull-request tagged with: PASSED, improvement, needs changes, type - frontend, type - summer coding - Request assigned
Hi @alishapapun, the changes look good. :thumbsup: I'm happy to merge this!
Before we do, could you please rebase and squash your commits into one commit? This helps keep the git changelog tidy. If you have never rebased before, see these two articles for help:
If you need additional guidance, don't hesitate to ask for help.
rebased onto ef0f0cc189e1e83614e74f48d125aea0aca7927c
Hey @jflory7 I have made the changes that you have asked for. Hope its what you expected. :)
Metadata Update from @jflory7: - Pull-request untagged with: needs changes
@jflory7 I will update the changes which is causing merge conflict and let you know.
Metadata Update from @jflory7: - Pull-request tagged with: needs changes
rebased onto 084675b8406ae38c7c2234d84b910c03fa106ad9
Hey @jflory7 , I have made the changes to no longer have the merge conflict. Could you please review it? I have the after image in #81 's last comment.
Thanks for working on this @alishapapun! This is ready to go. :100:
Merging! :clapper:
Pull-Request has been merged by jflory7
This Pull Request is regarding the issue #81.