Skip to content
This repository was archived by the owner on Jul 31, 2023. It is now read-only.

Conversation

@smalho01
Copy link

@smalho01 smalho01 commented Apr 7, 2022

No description provided.

Copy link

@KeeyanGhoreshi KeeyanGhoreshi left a comment

Choose a reason for hiding this comment

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

The console logs could be removed but it's not a big deal since DTR does log a lot in the console. It might also behoove us to use a variable either in the class or in the config for the endpoints here, to make life easier for future us, but it's not really a huge deal and works fine like this, too. Nothing to hold up the PR for.

@smalho01 smalho01 merged commit 200c951 into master Apr 15, 2022
@smalho01 smalho01 deleted the rems-129 branch November 2, 2022 17:45
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.

4 participants