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

feat: rename moonsama to moonriver #3904

Merged
merged 14 commits into from
Sep 9, 2022
Merged

feat: rename moonsama to moonriver #3904

merged 14 commits into from
Sep 9, 2022

Conversation

vikiival
Copy link
Member

@vikiival vikiival commented Sep 2, 2022

Thank you for your contribution to the KodaDot NFT gallery.
👇 _ Let's make a quick check before the contribution.

PR type

  • Bugfix
  • Feature
  • Refactoring

Related

What's new?

Updated

{
  "@kodadot1/vuex-options": "0.0.2"
}

Screenshot 2022-09-02 at 14 57 04

@vikiival vikiival requested review from a team as code owners September 2, 2022 12:56
@vikiival vikiival requested review from Jarsen136 and damskyftw and removed request for a team September 2, 2022 12:56
@netlify
Copy link

netlify bot commented Sep 2, 2022

Deploy Preview for koda-nuxt ready!

Name Link
🔨 Latest commit 06347a2
🔍 Latest deploy log https://app.netlify.com/sites/koda-nuxt/deploys/631b28790970d60008e16098
😎 Deploy Preview https://deploy-preview-3904--koda-nuxt.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@vikiival vikiival linked an issue Sep 2, 2022 that may be closed by this pull request
@yangwao yangwao changed the title feat: rename mooonbeam to moonriver feat: rename moonsama to moonriver Sep 2, 2022
@yangwao
Copy link
Member

yangwao commented Sep 2, 2022

image

@yangwao
Copy link
Member

yangwao commented Sep 2, 2022

POST https://app.gc.subsquid.io/beta/click/001-rc0/graphql net::ERR_FAILED

in #3884 we have 🤔

click: 'https://squid.subsquid.io/click/v/001/graphql',

@Jarsen136 Jarsen136 added the S-code-lgtm-✅ code review guild has reviewed this PR and it's code is approved label Sep 2, 2022
@yangwao
Copy link
Member

yangwao commented Sep 2, 2022

@yangwao
Copy link
Member

yangwao commented Sep 2, 2022

Something that could be tackled in

other than that seems it's up

image

@yangwao
Copy link
Member

yangwao commented Sep 2, 2022

https://deploy-preview-3904--koda-nuxt.netlify.app/movr/explore?tab=GALLERY&page=1

{
    "errors": [
        {
            "message": "Variable \"$orderBy\" of type \"[NFTEntityOrderByInput]\" used in position expecting type \"[NFTEntityOrderByInput!]\".",
            "extensions": {
                "code": "GRAPHQL_VALIDATION_FAILED",
                "exception": {
                    "stacktrace": [
                        "GraphQLError: Variable \"$orderBy\" of type \"[NFTEntityOrderByInput]\" used in position expecting type \"[NFTEntityOrderByInput!]\".",
                        "    at Object.leave (/squid/node_modules/graphql/validation/rules/VariablesInAllowedPositionRule.js:55:35)",
                        "    at Object.leave (/squid/node_modules/graphql/language/visitor.js:344:29)",
                        "    at Object.leave (/squid/node_modules/graphql/utilities/TypeInfo.js:390:21)",
                        "    at visit (/squid/node_modules/graphql/language/visitor.js:243:26)",
                        "    at validate (/squid/node_modules/graphql/validation/validate.js:69:24)",
                        "    at validate (/squid/node_modules/apollo-server-core/dist/requestPipeline.js:186:39)",
                        "    at processGraphQLRequest (/squid/node_modules/apollo-server-core/dist/requestPipeline.js:98:34)",
                        "    at processTicksAndRejections (node:internal/process/task_queues:96:5)",
                        "    at async processHTTPRequest (/squid/node_modules/apollo-server-core/dist/runHttpQuery.js:220:30)"
                    ]
                }
            }
        }
    ]
}

@yangwao
Copy link
Member

yangwao commented Sep 2, 2022

metadata on item detail?

https://deploy-preview-3904--koda-nuxt.netlify.app/movr/gallery/0x0dad866dc0c13fb8e4a91d1b5e83cf3a61d4cee2-7636

{
    "errors": [
        {
            "message": "Unknown type \"ID\".",
            "extensions": {
                "code": "GRAPHQL_VALIDATION_FAILED",
                "exception": {
                    "stacktrace": [
                        "GraphQLError: Unknown type \"ID\".",
                        "    at Object.NamedType (/squid/node_modules/graphql/validation/rules/KnownTypeNamesRule.js:57:29)",
                        "    at Object.enter (/squid/node_modules/graphql/language/visitor.js:323:29)",
                        "    at Object.enter (/squid/node_modules/graphql/utilities/TypeInfo.js:370:25)",
                        "    at visit (/squid/node_modules/graphql/language/visitor.js:243:26)",
                        "    at validate (/squid/node_modules/graphql/validation/validate.js:69:24)",
                        "    at validate (/squid/node_modules/apollo-server-core/dist/requestPipeline.js:186:39)",
                        "    at processGraphQLRequest (/squid/node_modules/apollo-server-core/dist/requestPipeline.js:98:34)",
                        "    at runMicrotasks (<anonymous>)",
                        "    at processTicksAndRejections (node:internal/process/task_queues:96:5)",
                        "    at async processHTTPRequest (/squid/node_modules/apollo-server-core/dist/runHttpQuery.js:220:30)"
                    ]
                }
            }
        }
    ]
}

@yangwao
Copy link
Member

yangwao commented Sep 2, 2022

Collection View

https://deploy-preview-3904--koda-nuxt.netlify.app/movr/collection/0xb6e9e605aa159017173caa6181c522db455f6661

{
    "errors": [
        {
            "message": "Unknown type \"ID\".",
            "extensions": {
                "code": "GRAPHQL_VALIDATION_FAILED",
                "exception": {
                    "stacktrace": [
                        "GraphQLError: Unknown type \"ID\".",
                        "    at Object.NamedType (/squid/node_modules/graphql/validation/rules/KnownTypeNamesRule.js:57:29)",
                        "    at Object.enter (/squid/node_modules/graphql/language/visitor.js:323:29)",
                        "    at Object.enter (/squid/node_modules/graphql/utilities/TypeInfo.js:370:25)",
                        "    at visit (/squid/node_modules/graphql/language/visitor.js:243:26)",
                        "    at validate (/squid/node_modules/graphql/validation/validate.js:69:24)",
                        "    at validate (/squid/node_modules/apollo-server-core/dist/requestPipeline.js:186:39)",
                        "    at processGraphQLRequest (/squid/node_modules/apollo-server-core/dist/requestPipeline.js:98:34)",
                        "    at processTicksAndRejections (node:internal/process/task_queues:96:5)",
                        "    at async processHTTPRequest (/squid/node_modules/apollo-server-core/dist/runHttpQuery.js:220:30)"
                    ]
                }
            }
        }
    ]
}
{
    "errors": [
        {
            "message": "Unknown type \"ID\".",
            "extensions": {
                "code": "GRAPHQL_VALIDATION_FAILED",
                "exception": {
                    "stacktrace": [
                        "GraphQLError: Unknown type \"ID\".",
                        "    at Object.NamedType (/squid/node_modules/graphql/validation/rules/KnownTypeNamesRule.js:57:29)",
                        "    at Object.enter (/squid/node_modules/graphql/language/visitor.js:323:29)",
                        "    at Object.enter (/squid/node_modules/graphql/utilities/TypeInfo.js:370:25)",
                        "    at visit (/squid/node_modules/graphql/language/visitor.js:243:26)",
                        "    at validate (/squid/node_modules/graphql/validation/validate.js:69:24)",
                        "    at validate (/squid/node_modules/apollo-server-core/dist/requestPipeline.js:186:39)",
                        "    at processGraphQLRequest (/squid/node_modules/apollo-server-core/dist/requestPipeline.js:98:34)",
                        "    at runMicrotasks (<anonymous>)",
                        "    at processTicksAndRejections (node:internal/process/task_queues:96:5)",
                        "    at async processHTTPRequest (/squid/node_modules/apollo-server-core/dist/runHttpQuery.js:220:30)"
                    ]
                }
            }
        }
    ]
}

@vikiival
Copy link
Member Author

vikiival commented Sep 3, 2022

Screenshot 2022-09-03 at 16 57 53

@vikiival
Copy link
Member Author

vikiival commented Sep 3, 2022

Best would be to downgrade click

@yangwao yangwao marked this pull request as draft September 5, 2022 11:00
auto-merge was automatically disabled September 5, 2022 11:00

Pull request was converted to draft

@yangwao yangwao added the A-moonriver issues related to MoonRiver network label Sep 7, 2022
@yangwao
Copy link
Member

yangwao commented Sep 9, 2022

one random
image

otherwise, seems working?

image

@yangwao yangwao marked this pull request as ready for review September 9, 2022 08:50
@yangwao yangwao requested review from petersopko and removed request for damskyftw September 9, 2022 08:50
@yangwao
Copy link
Member

yangwao commented Sep 9, 2022

@petersopko let's check what's missing and what's needs to be done and update

@petersopko
Copy link
Contributor

  1. collections don't show image previews+ errors in console
    image

  2. same for landing / curated lists
    image

  3. searching through explorer is hard / takes too long to load (sometimes it just gets stuck)
    image

@petersopko
Copy link
Contributor

holders/flippers:
image

@vikiival
Copy link
Member Author

vikiival commented Sep 9, 2022

Balance is fixed in #3926

@vikiival
Copy link
Member Author

vikiival commented Sep 9, 2022

@yangwao the issue is already there for 3 months #3338

@petersopko petersopko mentioned this pull request Sep 9, 2022
@petersopko
Copy link
Contributor

let's check what's missing and what's needs to be done and update
#3647
accordingly with sub-issues

@yangwao @vikiival I'm kind of confused about this PR, are we just renaming moonsama to moonriver, or fixing everything that's wrong with MOVR?

@vikiival
Copy link
Member Author

vikiival commented Sep 9, 2022

Maybe just a bad issue #3902

@yangwao
Copy link
Member

yangwao commented Sep 9, 2022

are we just renaming moonsama to moonriver,

yes

or fixing everything that's wrong with MOVR?

also fixing rest once it's this minimum viable working version merged so others can continue

@codeclimate
Copy link

codeclimate bot commented Sep 9, 2022

Code Climate has analyzed commit 06347a2 and detected 1 issue on this pull request.

Here's the issue category breakdown:

Category Count
Duplication 1

View more on Code Climate.

@yangwao
Copy link
Member

yangwao commented Sep 9, 2022

fix tests here or merge it and fix them afterwards?

wyt @petersopko

@petersopko
Copy link
Contributor

fix tests here or merge it and fix them afterwards?

wyt @petersopko

image

the failing tests here are the usual media tests

I've hit re-run but regardless, I think it's save to merge

@yangwao yangwao merged commit 3c170f1 into main Sep 9, 2022
@yangwao yangwao deleted the can-we-have-some-movr branch September 9, 2022 13:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-moonriver issues related to MoonRiver network S-code-lgtm-✅ code review guild has reviewed this PR and it's code is approved
Projects
None yet
Development

Successfully merging this pull request may close these issues.

First batch of MoonRiver contracts
4 participants