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

Submodule Registry Integration #215

Open
wants to merge 15 commits into
base: development
Choose a base branch
from

Conversation

bbroeking
Copy link
Collaborator

@bbroeking bbroeking commented Dec 5, 2022

two more things to check

  1. vercel needs to be able to use the submodule in the build (need to add some permissions)
  2. add the submodule to the build such that the UI can use the imgs to render (not sure what this path needs to be)

BowTiedHeron and others added 12 commits December 5, 2022 09:13
* market verified creator wip

* add temp debt fix | remove logs | add constants for validation | add remainder markets for future risk funcs

* adjust market constants

* wip loanhelpers

* add verified creator in validation

* filter empty pos

* verfied creators

* update honey sdk

* remove unnecessary params

* remove unnecessary params and code

* add description

* interestrate wip

* set max ratio to 100%

* remove logs

* downgrade package @jup-ag/react-hook

* remove logs

* add logs

* add discounted base borrow rate

* add market id for getinterest rate func

* add collection id for get interest rate

* add default market for interest rate and todo before dashboard goes live to update func. with dynamic market ids

* add discount borrow rate for lifinity and atadians

* add comments

* change func. name

* remove unused code | add coments | remove dependencies from array

* remove log and comment

* allow column to open when no wallet is connected

* add more comments |comment setup

* remove code | add comments | add verified creator

* add verified creator for deposit nft func

* add local log

* remove log

* update UI after deposit withdraw

* local testing with fetchPositionBids

* add filter function to filter out all nfts that do not include verified creator

* adjust image array

* remove commented statements

* remove commented code | add comments

* add comments

* remove non used imports | add comments

* add comments

* add comments | remove duplication

* remove imports | add comments

* add comments

* remove unused code

* adjust and add types

* remove unused code | add comments | add creators

* add comment

* add type | rename item to NFT | add comment

* adjust comment

* add comments | remove imports and unused code | remove await

* add comments | remove unused imports

* remove unused code

* add constants for BSL

* add bsl market id

* remove unused imports

* add BSL to markets

* add BSL to helper functions

* adjust imports | move constants from constants to helpers/marketHelpers | remove unused imports | add comment

* add log

* fix interest rate

* fix interest rate

* remove bsl from markets

* adjust path

* adjust import

* add constants object

* adjust set market func

* adjust imports | func param call | constants and market setup

Co-authored-by: Bowtiedpyro <[email protected]>
@vercel
Copy link

vercel bot commented Dec 5, 2022

@bbroeking is attempting to deploy a commit to the Honey Labs Team on Vercel.

To accomplish this, @bbroeking needs to request access to the Team.

Afterwards, an owner of the Team is required to accept their membership request.

If you're already a member of the respective Vercel Team, make sure that your Personal Vercel Account is connected to your GitHub account.

@vercel
Copy link

vercel bot commented Dec 5, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
governance-test-env ✅ Ready (Inspect) Visit Preview Dec 12, 2022 at 6:19PM (UTC)
honey-frontend-redesign ✅ Ready (Inspect) Visit Preview Dec 12, 2022 at 6:19PM (UTC)

@vercel
Copy link

vercel bot commented Dec 5, 2022

An owner of the Honey Labs Team on Vercel accepted @bbroeking's request to join.

@bbroeking's commit is now being deployed.

@bowtiedpyro
Copy link
Collaborator

Looks good @bbroeking - we can merge this in.

@bbroeking
Copy link
Collaborator Author

Looks good @bbroeking - we can merge this in.

@bowtiedpyro I'll resolve the merge conflicts, but I think there's still 2 things to address.

Not sure how to include the git submodule in the next build such that it's images are useable.

Also I think some of the new render logic is breaking the lend page

on:
# Triggers the workflow on push or pull request events but only for the "main" branch
push:
branches: ['main', 'feature/submodule-registry-integ']
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

will need to add any other branches

on:
# Triggers the workflow on push or pull request events but only for the "main" branch
push:
branches: ['main']
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

here

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.

3 participants