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

add support for NFT transfers, burns, and acceptance of transfers #545

Merged
merged 3 commits into from
Jan 5, 2022

Conversation

lazynina
Copy link
Member

@lazynina lazynina commented Jan 3, 2022

What's been done:

  1. Add modals for transfer NFT, burn NFT, and accept NFT transfer
  2. Show "Pending" in the Owners table if owner has not accepted transfer yet
  3. Update NFTs view on user's profile to have Transferrable and Pending Transfers views.
  4. Make re-usable serial number selector
  5. Update close modal listeners

Pending owner text
image

Feed post dropdown updates
image

Pending Transfer View
image

Transferrable View
image

Transfer NFT Modal
image

Burn NFT modal
image

Accept Transfer Modal
image

@lazynina lazynina requested a review from a team as a code owner January 3, 2022 21:26
@@ -215,6 +216,7 @@
<i class="fas fa-check-circle fa-md align-middle"></i>
</span>
</div>
<div class="fc-red" *ngIf="owner.IsPending"><i>Pending</i></div>
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we want this to be red? Maybe just gray or something?

Copy link
Member Author

Choose a reason for hiding this comment

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

yeah gray is less intimidating
image

selector: "nft-select-serial-number",
templateUrl: "./nft-select-serial-number.component.html",
})
export class NftSelectSerialNumberComponent implements OnInit {
Copy link
Contributor

Choose a reason for hiding this comment

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

Nice!

@lazynina lazynina merged commit f83d7ce into main Jan 5, 2022
@lazynina lazynina deleted the ln/transfer-burn-accept-nfts branch January 5, 2022 16:43
piratekev added a commit to TheBitgram/frontend that referenced this pull request Jan 19, 2022
* add notification transformation for NFT Transfers, fix console error when there are no fees in TransactionFeeMap (deso-protocol#526)

* Fix calculation of ETH exchange rate including fee (deso-protocol#530)

* add supply-stats route to show total supply and rich list (deso-protocol#531)

* Fix preview and home screen icons for DeSo (deso-protocol#532)

* Add support for managing sign-up bonus configurations (deso-protocol#529)

* save current progress on updating admin panel

* add support for modifying the sign up bonus config for a single country, update to use default jumio USD cents instead of DeSo nanos

* refresh country bonuses after updating default jumio USD cents

* add tooltip disclaimer about free DESO amount

* update copy

* simplify loops in GetMessages that handles encryption/decryption (deso-protocol#533)

* simplify loops in GetMessages that handles encryption/decryption

* message -> Message

* Fix admin jumio checkboxes (deso-protocol#534)

* use country sign up bonus config inferred from IP address when computing referral amount to display for sign up bonus (deso-protocol#535)

* use flatMap to flatten array of message before decryption (deso-protocol#536)

* Upload referral csv directly instead of parsing on the frontend (deso-protocol#537)

* top diamonded list fix (deso-protocol#480)

* use altumbase for daily gainers leaderboard (deso-protocol#538)

* use altumbase for daily gainers leaderboard

* fix import styling

* [stable] Release 1.2.9

* add disclaimer on referrals page about amounts varying by locality of ID AND add support for setting default kickback amount (deso-protocol#539)

* add disclaimer on referrals page about amounts varying by locality of ID

* add support for updating the default kickback amount for referrers

* add referral code, jumio starter DESO txn Hash, and referrer DeSo Txn hash to User Admin Data (deso-protocol#540)

* make referral link relative to window origin (deso-protocol#541)

* Ln/count keys with deso (deso-protocol#542)

* save current progress

* update some styling on the supply monitoring page

* update supply stats page to show count of keys holding DESO

* Update src/app/supply-monitoring-stats-page/supply-monitoring-stats/supply-monitoring-stats.component.html

* fix admin panel jumio kickback usd cents (deso-protocol#544)

* add support for NFT transfers, burns, and acceptance of transfers (deso-protocol#545)

* add support for NFT transfers, burns, and acceptance of transfers

* fix styling

* change font color to gray for pending ownership

* add fas class to fix issue with icons not appearing in select serial number component (deso-protocol#547)

* Buy Now NFTs and NFT Splits (deso-protocol#546)

* [stable] Release 2.0.0 (deso-protocol#550)

* [stable] Release 2.0.1

Co-authored-by: Lazy Nina <81658138+lazynina@users.noreply.github.com>
Co-authored-by: NikolaiL <nikolai1973@gmail.com>
Co-authored-by: maebeam <maebeam0@gmail.com>
Co-authored-by: diamondhands0 <81935176+diamondhands0@users.noreply.github.com>
Co-authored-by: diamondhands <diamondhands@bitcloutdev.com>
piratekev added a commit to TheBitgram/frontend that referenced this pull request May 18, 2022
* add notification transformation for NFT Transfers, fix console error when there are no fees in TransactionFeeMap (deso-protocol#526)

* Fix calculation of ETH exchange rate including fee (deso-protocol#530)

* add supply-stats route to show total supply and rich list (deso-protocol#531)

* Fix preview and home screen icons for DeSo (deso-protocol#532)

* Add support for managing sign-up bonus configurations (deso-protocol#529)

* save current progress on updating admin panel

* add support for modifying the sign up bonus config for a single country, update to use default jumio USD cents instead of DeSo nanos

* refresh country bonuses after updating default jumio USD cents

* add tooltip disclaimer about free DESO amount

* update copy

* simplify loops in GetMessages that handles encryption/decryption (deso-protocol#533)

* simplify loops in GetMessages that handles encryption/decryption

* message -> Message

* Fix admin jumio checkboxes (deso-protocol#534)

* use country sign up bonus config inferred from IP address when computing referral amount to display for sign up bonus (deso-protocol#535)

* use flatMap to flatten array of message before decryption (deso-protocol#536)

* Upload referral csv directly instead of parsing on the frontend (deso-protocol#537)

* top diamonded list fix (deso-protocol#480)

* use altumbase for daily gainers leaderboard (deso-protocol#538)

* use altumbase for daily gainers leaderboard

* fix import styling

* [stable] Release 1.2.9

* add disclaimer on referrals page about amounts varying by locality of ID AND add support for setting default kickback amount (deso-protocol#539)

* add disclaimer on referrals page about amounts varying by locality of ID

* add support for updating the default kickback amount for referrers

* add referral code, jumio starter DESO txn Hash, and referrer DeSo Txn hash to User Admin Data (deso-protocol#540)

* make referral link relative to window origin (deso-protocol#541)

* Ln/count keys with deso (deso-protocol#542)

* save current progress

* update some styling on the supply monitoring page

* update supply stats page to show count of keys holding DESO

* Update src/app/supply-monitoring-stats-page/supply-monitoring-stats/supply-monitoring-stats.component.html

* fix admin panel jumio kickback usd cents (deso-protocol#544)

* add support for NFT transfers, burns, and acceptance of transfers (deso-protocol#545)

* add support for NFT transfers, burns, and acceptance of transfers

* fix styling

* change font color to gray for pending ownership

* add fas class to fix issue with icons not appearing in select serial number component (deso-protocol#547)

* Buy Now NFTs and NFT Splits (deso-protocol#546)

* [stable] Release 2.0.0 (deso-protocol#550)

* [stable] Release 2.0.1

* nft notification enhancements (deso-protocol#551)

* save current progress on notifications

* Update NFT notifications and fix some small bugs

* make global vars private again

* remove console logging

* Ln/update sell nft modal (deso-protocol#554)

* show additional royalties in sell nft modal

* remove service fee which is no longer user

* Messages V3 (deso-protocol#543)

* Messages V3

* V3 support

* Messages V3 final

* Add comments & fix version

* Fix circular dependancies

* Revert "Messages V3 (deso-protocol#543)" (deso-protocol#555)

This reverts commit 9d98ea7.

* if get user metadata throws an error, swallow it and return null (deso-protocol#556)

* [stable] Release 2.0.2

* Ln/dao coins (deso-protocol#548)

* save current progress on DAO coin UI

* add support for transferring DAO coins in the UI, add inputs for other fields for DAO coin transaction

* add burn support, add DAO coin tab to profile, use number abbreviation to keep DAO coin numbers manageable, move utility func for parsing hex balances to global vars

* add some frontend validation in transfer modal

* fix up DAO modals - add balances and validations, hit isHodling endpoint if transfer restriction status is DAO Members only

* fix alignment on DAO coin page

* disable mint and burn if the amount is less than or equal to 0

* add notifications for DAO coin txns

* add sweet alerts before DAO actions, only show profile owner if transfer restriction status is profile owner only and logged in user is not profile owner

* address TGS feedback

* Fix errors when user does not have DAO coin yet

* Messages v3 (deso-protocol#557)

* Messages V3

* V3 support

* Messages V3 final

* Add comments & fix version

* Fix circular dependancies

* Add senderGroupKeyName to TransferNFT

* update Query ETH RPC to remove JWT requirement (deso-protocol#558)

* Add support for Pearl node (deso-protocol#561)

* Posts/Users pages are blank when user not logged in (deso-protocol#560)

* fix: metadata fails when user not logged in

this additional checks makes sure verification / node.deso.org metadata is only requested if loggedInUserPublicKey is ok. Currently it requests metadata from `https://node.deso.org/api/v0/get-user-metadata/undefined` which returns status 404 error and prevents post/user pages from showing the content.

* align with diamondapp

use same approach as DiamondApp

* [stable] Release 2.0.4 (deso-protocol#562)

* display media content in replies (deso-protocol#564)

* allow media when creating comments (deso-protocol#565)

* Add support for twitter images.. (deso-protocol#568)

* Add support for twitter images..

this will show images hosted on twitter (they have link like pbs.twing.com ex: https://pbs.twimg.com/media/FO9LbSjaQAEFE26.jpg)

before 
![image](https://user-images.githubusercontent.com/55331140/160470352-bde7f317-fcc7-4680-9180-f9487a7892bc.png)

after
![image](https://user-images.githubusercontent.com/55331140/160470432-5168c8a3-ad78-44e9-a147-3afc1fd11803.png)

will be helpful for those projects which are building bridge between twitter and deso.

* Update Caddyfile

Co-authored-by: Lazy Nina <81658138+lazynina@users.noreply.github.com>

* added Mousai embedding support to frontend (deso-protocol#563)

* Added support for embedding Mousai streaming link in frontend

* Fixed prettier/prettier issue

* Added tests for Mousai embedding functionality

* improved regex matching for mousai links

* added the mousai's url in Caddyfile

* Removed the test-cases that prevent tests from running properly

Co-authored-by: Lazy Nina <81658138+lazynina@users.noreply.github.com>

* Replaced `jasmin.arrayContaining` call with a more logically sounding alternative (based on the available variables)

Co-authored-by: Lazy Nina <81658138+lazynina@users.noreply.github.com>

* Updated `regex` based on the suggestion provided by @lazynina

Co-authored-by: Lazy Nina <81658138+lazynina@users.noreply.github.com>

* Updated `regex` based on the suggestion provided by @lazynina

Co-authored-by: Lazy Nina <81658138+lazynina@users.noreply.github.com>

* Added proper height to Mousai link

* Update src/lib/services/embed-url-parser-service/embed-url-parser-service.ts

Co-authored-by: Lazy Nina <81658138+lazynina@users.noreply.github.com>

* Update src/lib/services/embed-url-parser-service/embed-url-parser-service.ts

Co-authored-by: Lazy Nina <81658138+lazynina@users.noreply.github.com>

Co-authored-by: Lazy Nina <81658138+lazynina@users.noreply.github.com>

* [stable] Release 2.1.0 (deso-protocol#569)

* update reporting links (deso-protocol#571)

* update reporting links

* add /content to path

* Remove jumio messaging and referrals (deso-protocol#573)

* [stable] Release 2.2.0

* [stable] Release 2.2.1

Bumping release to fix a CI issue

* pull in v2.2.1

Co-authored-by: Lazy Nina <81658138+lazynina@users.noreply.github.com>
Co-authored-by: NikolaiL <nikolai1973@gmail.com>
Co-authored-by: maebeam <maebeam0@gmail.com>
Co-authored-by: diamondhands0 <81935176+diamondhands0@users.noreply.github.com>
Co-authored-by: diamondhands <diamondhands@bitcloutdev.com>
Co-authored-by: Piotr Nojszewski <29924594+AeonSw4n@users.noreply.github.com>
Co-authored-by: Tijno (@tijn on Deso) <69529928+tijno@users.noreply.github.com>
Co-authored-by: ItsAditya.eth <55331140+AdityaChaudhary0005@users.noreply.github.com>
Co-authored-by: Farsad Fakhim <fakhim.farsad@gmail.com>
piratekev added a commit to TheBitgram/frontend that referenced this pull request Jul 28, 2022
* add notification transformation for NFT Transfers, fix console error when there are no fees in TransactionFeeMap (deso-protocol#526)

* Fix calculation of ETH exchange rate including fee (deso-protocol#530)

* add supply-stats route to show total supply and rich list (deso-protocol#531)

* Fix preview and home screen icons for DeSo (deso-protocol#532)

* Add support for managing sign-up bonus configurations (deso-protocol#529)

* save current progress on updating admin panel

* add support for modifying the sign up bonus config for a single country, update to use default jumio USD cents instead of DeSo nanos

* refresh country bonuses after updating default jumio USD cents

* add tooltip disclaimer about free DESO amount

* update copy

* simplify loops in GetMessages that handles encryption/decryption (deso-protocol#533)

* simplify loops in GetMessages that handles encryption/decryption

* message -> Message

* Fix admin jumio checkboxes (deso-protocol#534)

* use country sign up bonus config inferred from IP address when computing referral amount to display for sign up bonus (deso-protocol#535)

* use flatMap to flatten array of message before decryption (deso-protocol#536)

* Upload referral csv directly instead of parsing on the frontend (deso-protocol#537)

* top diamonded list fix (deso-protocol#480)

* use altumbase for daily gainers leaderboard (deso-protocol#538)

* use altumbase for daily gainers leaderboard

* fix import styling

* [stable] Release 1.2.9

* add disclaimer on referrals page about amounts varying by locality of ID AND add support for setting default kickback amount (deso-protocol#539)

* add disclaimer on referrals page about amounts varying by locality of ID

* add support for updating the default kickback amount for referrers

* add referral code, jumio starter DESO txn Hash, and referrer DeSo Txn hash to User Admin Data (deso-protocol#540)

* make referral link relative to window origin (deso-protocol#541)

* Ln/count keys with deso (deso-protocol#542)

* save current progress

* update some styling on the supply monitoring page

* update supply stats page to show count of keys holding DESO

* Update src/app/supply-monitoring-stats-page/supply-monitoring-stats/supply-monitoring-stats.component.html

* fix admin panel jumio kickback usd cents (deso-protocol#544)

* add support for NFT transfers, burns, and acceptance of transfers (deso-protocol#545)

* add support for NFT transfers, burns, and acceptance of transfers

* fix styling

* change font color to gray for pending ownership

* add fas class to fix issue with icons not appearing in select serial number component (deso-protocol#547)

* Buy Now NFTs and NFT Splits (deso-protocol#546)

* [stable] Release 2.0.0 (deso-protocol#550)

* [stable] Release 2.0.1

* nft notification enhancements (deso-protocol#551)

* save current progress on notifications

* Update NFT notifications and fix some small bugs

* make global vars private again

* remove console logging

* Ln/update sell nft modal (deso-protocol#554)

* show additional royalties in sell nft modal

* remove service fee which is no longer user

* Messages V3 (deso-protocol#543)

* Messages V3

* V3 support

* Messages V3 final

* Add comments & fix version

* Fix circular dependancies

* Revert "Messages V3 (deso-protocol#543)" (deso-protocol#555)

This reverts commit 9d98ea7.

* if get user metadata throws an error, swallow it and return null (deso-protocol#556)

* [stable] Release 2.0.2

* Ln/dao coins (deso-protocol#548)

* save current progress on DAO coin UI

* add support for transferring DAO coins in the UI, add inputs for other fields for DAO coin transaction

* add burn support, add DAO coin tab to profile, use number abbreviation to keep DAO coin numbers manageable, move utility func for parsing hex balances to global vars

* add some frontend validation in transfer modal

* fix up DAO modals - add balances and validations, hit isHodling endpoint if transfer restriction status is DAO Members only

* fix alignment on DAO coin page

* disable mint and burn if the amount is less than or equal to 0

* add notifications for DAO coin txns

* add sweet alerts before DAO actions, only show profile owner if transfer restriction status is profile owner only and logged in user is not profile owner

* address TGS feedback

* Fix errors when user does not have DAO coin yet

* Messages v3 (deso-protocol#557)

* Messages V3

* V3 support

* Messages V3 final

* Add comments & fix version

* Fix circular dependancies

* Add senderGroupKeyName to TransferNFT

* update Query ETH RPC to remove JWT requirement (deso-protocol#558)

* Add support for Pearl node (deso-protocol#561)

* Posts/Users pages are blank when user not logged in (deso-protocol#560)

* fix: metadata fails when user not logged in

this additional checks makes sure verification / node.deso.org metadata is only requested if loggedInUserPublicKey is ok. Currently it requests metadata from `https://node.deso.org/api/v0/get-user-metadata/undefined` which returns status 404 error and prevents post/user pages from showing the content.

* align with diamondapp

use same approach as DiamondApp

* [stable] Release 2.0.4 (deso-protocol#562)

* display media content in replies (deso-protocol#564)

* allow media when creating comments (deso-protocol#565)

* Add support for twitter images.. (deso-protocol#568)

* Add support for twitter images..

this will show images hosted on twitter (they have link like pbs.twing.com ex: https://pbs.twimg.com/media/FO9LbSjaQAEFE26.jpg)

before 
![image](https://user-images.githubusercontent.com/55331140/160470352-bde7f317-fcc7-4680-9180-f9487a7892bc.png)

after
![image](https://user-images.githubusercontent.com/55331140/160470432-5168c8a3-ad78-44e9-a147-3afc1fd11803.png)

will be helpful for those projects which are building bridge between twitter and deso.

* Update Caddyfile

Co-authored-by: Lazy Nina <81658138+lazynina@users.noreply.github.com>

* added Mousai embedding support to frontend (deso-protocol#563)

* Added support for embedding Mousai streaming link in frontend

* Fixed prettier/prettier issue

* Added tests for Mousai embedding functionality

* improved regex matching for mousai links

* added the mousai's url in Caddyfile

* Removed the test-cases that prevent tests from running properly

Co-authored-by: Lazy Nina <81658138+lazynina@users.noreply.github.com>

* Replaced `jasmin.arrayContaining` call with a more logically sounding alternative (based on the available variables)

Co-authored-by: Lazy Nina <81658138+lazynina@users.noreply.github.com>

* Updated `regex` based on the suggestion provided by @lazynina

Co-authored-by: Lazy Nina <81658138+lazynina@users.noreply.github.com>

* Updated `regex` based on the suggestion provided by @lazynina

Co-authored-by: Lazy Nina <81658138+lazynina@users.noreply.github.com>

* Added proper height to Mousai link

* Update src/lib/services/embed-url-parser-service/embed-url-parser-service.ts

Co-authored-by: Lazy Nina <81658138+lazynina@users.noreply.github.com>

* Update src/lib/services/embed-url-parser-service/embed-url-parser-service.ts

Co-authored-by: Lazy Nina <81658138+lazynina@users.noreply.github.com>

Co-authored-by: Lazy Nina <81658138+lazynina@users.noreply.github.com>

* [stable] Release 2.1.0 (deso-protocol#569)

* update reporting links (deso-protocol#571)

* update reporting links

* add /content to path

* Remove jumio messaging and referrals (deso-protocol#573)

* [stable] Release 2.2.0

* [stable] Release 2.2.1

Bumping release to fix a CI issue

* Add support for resetting by phone number for super admins (deso-protocol#570)

* Adjust DAO Coin base units to be 10^18 (deso-protocol#574)

* Adjust DAO Coin base units to be 10^18

* Fix dao coin base units

* Add panel to allow configuration of hot feed (deso-protocol#576)

* Fix Max send bug (deso-protocol#577)

* Added # (number sign) support for parsing Mousai links (deso-protocol#578)

* Fix mousai embed to escape # character in URL (deso-protocol#579)

* Updates to feeds (deso-protocol#583)

* Add hot feed post multiplier for individual posts (deso-protocol#584)

* Support for piñata IPFS gateways (deso-protocol#580)

* Formatting + remove free deso (deso-protocol#585)

* Formatting + remove free deso

* Removed additional free deso text

* remove duplicate fns

Co-authored-by: Lazy Nina <81658138+lazynina@users.noreply.github.com>
Co-authored-by: NikolaiL <nikolai1973@gmail.com>
Co-authored-by: maebeam <maebeam0@gmail.com>
Co-authored-by: diamondhands0 <81935176+diamondhands0@users.noreply.github.com>
Co-authored-by: diamondhands <diamondhands@bitcloutdev.com>
Co-authored-by: Piotr Nojszewski <29924594+AeonSw4n@users.noreply.github.com>
Co-authored-by: Tijno (@tijn on Deso) <69529928+tijno@users.noreply.github.com>
Co-authored-by: ItsAditya.eth <55331140+AdityaChaudhary0005@users.noreply.github.com>
Co-authored-by: Farsad Fakhim <fakhim.farsad@gmail.com>
Co-authored-by: triplegreenshell <94129308+triplegreenshell@users.noreply.github.com>
Co-authored-by: Martijn van Halen <martijn.vanhalen@gmail.com>
Co-authored-by: TylerFischer@Deso <98359380+DeSoDog@users.noreply.github.com>
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