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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Asylum: milestone 1 #438

Merged
merged 2 commits into from May 23, 2022
Merged

Asylum: milestone 1 #438

merged 2 commits into from May 23, 2022

Conversation

YaremenkoM
Copy link
Contributor

Milestone Delivery Checklist

Link to the application pull request: w3f/Grants-Program#805

@LastSymbol0
Copy link

Hey!

I know that web3 foundation is pretty busy right now as you have a lot of deliveries.
But just let me know if there are any blockers for you to review our request and I would be glad to help you!

@LastSymbol0
Copy link

Hi @semuelle !

Tagging you because you were in conversation on our application PR.
Could you please help me with an estimation for that delivery review?

@semuelle
Copy link
Member

Sorry for the delay, @LastSymbol0 & @YaremenkoM. I will look into it tomorrow.

@semuelle
Copy link
Member

Hey @YaremenkoM & @LastSymbol0. I had a look at your delivery today. Sorry again for the delay here.

Code and documentation look very good so far, but I noticed that you didn't submit a Docker file/image or an article. They are more or less mandatory for all milestones, at least where appropriate (M1s often come without articles for understandable reasons). Could you also provide links for deliverables 1.1 to 1.3, either to the most important code sections for these functionalities or references to it in the testing guide?

Add 1.1 - 1.3 deliveries
@LastSymbol0
Copy link

Hey @semuelle , thanks for your feedback!

We've just submitted all requested updates:

  • added Dockerfiles to our repositories (and docker-compose to UI repo) and mentioned them in the testing guide
  • extended the testing guide regarding multiple 3d and 2d interpretations
  • added deliverables 1.1 to 1.3 and links to the particular sections of the testing guide

Looking forward to the review!

@semuelle
Copy link
Member

Thanks for the update, @LastSymbol0, and sorry for the late reply.

I had some trouble getting the setup to complete. host.docker.internal seems to be a Windows specific variable. Also, the manual setup instructions are missing the yarn and yarn start steps. I got around those issues, but it would be great if you could clear those up for the next milestone. All the functionality seems to be there, only I cannot mint any NFTs. I was able to mint them through polkadotjs, but clicking the mint item button does nothing. Am I missing something?

Other than that I am quite happy with the code and material. The UI is really sleek, great job!

@LastSymbol0
Copy link

Hi @semuelle , thanks for your comments!

We fixed issues with docker and tested them, now everything should work well. Also, the documentation was reorganized, considering your point about scripts (yarn start and yarn seed)

Speaking about minting items - as the application says - functionality related to the items is part of the second milestone, so we do not work on it within the first one. To clarify this moment we've also added the notification to the app by pressing the "Mint" button.

@semuelle
Copy link
Member

Thanks for the update, @LastSymbol0. The grant document actually mentions minting in milestone 1, but I see now what you are saying.

Your milestone is hereby accepted. Congratulations. :) I will forward your invoice for processing. Evaluation notes here.

@semuelle semuelle merged commit 2fa9e0b into w3f:master May 23, 2022
@LastSymbol0
Copy link

Amazing, thanks for your review!

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.

None yet

3 participants