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

[#84] Add point inspect for mosaics #96

Merged
merged 1 commit into from
Aug 15, 2023

Conversation

sandrahoang686
Copy link
Collaborator

@sandrahoang686 sandrahoang686 commented Aug 11, 2023

Summary:

Fixes or Addresses Issue #: [ticket number & link]

#84

NOTE: I hope to come back in here to quickly add testing ability but I need to figure out how to mock the request, I will do that in a separate PR so this feature is not blocked

DEMO in Local: https://www.loom.com/share/8d5305834a714557bccf9e60d810d792

Checklist before requesting a review:

  • My code follows the guidelines of this project
  • I performed a self-review of my code changes
  • I have added my changes to the change log (HISTORY.rst)
  • I have completed spell-checking, removed unnecessary print statements & commented-out code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • New and existing unit tests pass locally with my changes

@sandrahoang686 sandrahoang686 requested a review from emmalu August 11, 2023 16:32
@sandrahoang686 sandrahoang686 merged commit fe8386f into main Aug 15, 2023
@wildintellect wildintellect deleted the feature/add-point-inspect-mosaics branch September 27, 2023 22:12
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