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.

Sign up for GitHub

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

Jump to bottom

NFTs2Me.com dApp submission #12

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

NFTs2Me.com dApp submission #12

wants to merge 1 commit into from

Conversation

nfts2me
Copy link

@nfts2me nfts2me commented Apr 20, 2023

Submission for NFTs2Me

@ivopaunov
Copy link
Member

@nfts2me can you please add the config only in src/catalogs/wallet-gnosis.applist.json there is no need to be in other files. Looks like it works with gnosis safe but there is some blockage with the iframe. forceInternal should do the work if there is no cors or something else.

@nfts2me
Copy link
Author

nfts2me commented May 2, 2023

@nfts2me can you please add the config only in src/catalogs/wallet-gnosis.applist.json there is no need to be in other files. Looks like it works with gnosis safe but there is some blockage with the iframe. forceInternal should do the work if there is no cors or something else.

Thank you.
I'll do the change. Before that:

Leena8686
Leena8686 approved these changes May 3, 2023
View reviewed changes
src/catalogs/ambire-wallet-gnosis-default.applist.json
"url": "https://nfts2me.com/",
"name": "NFTs2Me",
"iconUrl": "https://nfts2me.com/assets/images/logo.svg",
"description": "Multichain user-friendly comprehensive platform to create, deploy and manage your NFT collection and community, 100% free with advanced functionalities.",
Copy link

Choose a reason for hiding this comment

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

deploy

@Leena8686
Copy link

Pull request.

@JIOjosBG
Copy link
Member

We have updated the json files to include applicationType array for all dapps. Please fix the merge conflicts and add applicationType: ['mobile','web'] to your dapp

Leena8686
Leena8686 approved these changes Sep 13, 2023
View reviewed changes
Copy link

@Leena8686 Leena8686 left a comment

Choose a reason for hiding this comment

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

AmbireTech:main from nft2me:main

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Leena8686 Leena8686 Leena8686 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@nfts2me @ivopaunov @Leena8686 @JIOjosBG @agsola