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

Alert pop-up : Metamask installation #62

Open
Supriya2922 opened this issue Mar 7, 2022 · 10 comments
Open

Alert pop-up : Metamask installation #62

Supriya2922 opened this issue Mar 7, 2022 · 10 comments
Assignees
Labels
assigned GSSoC22 Girl Script Summer of Code 2022 Level1

Comments

@Supriya2922
Copy link

Supriya2922 commented Mar 7, 2022

An alert pop-up which appears only if metamask is not installed
Why it's essential ?
Metamask extension installed in the web browser is very important in order to run this project.
image
I would like to work on this.

@megabyte0x
Copy link
Owner

Hey @Supriya2922,
Thanks for the suggestion.
We would like to make an alert for this, but it would be better if you make a toast instead of an alert.
If you have any other query, please let us know.

Thanks for contributing
Megabyte

@pranay202
Copy link

I want to work on this. Please assign me this issue.

@raysummee
Copy link
Collaborator

@pranay202 I have assigned you this issue. You can start working on this now.
Please try to add a toast instead of an alert if possible.

@raysummee raysummee added assigned GSSoC22 Girl Script Summer of Code 2022 Level1 labels Mar 31, 2022
@pranay202
Copy link

Ok. Please tell me about file structure.
My suggestion: client/context/TransactionContext.jsx

@raysummee
Copy link
Collaborator

we're having this kind of structure for UI:

for page:

client/pages/pagename/pagename.js
client/pages/pagename/pagename.module.css

for components:

client/pages/components/componentname/componentname.js
client/pages/components/componentname/componentname.module.css

We've added component to the pages so that we can inspect the component easily. Later we'll shift components to the relevant folder

@pranay202 pranay202 removed their assignment Apr 3, 2022
@pranay202
Copy link

Due to my busy schedule, I'm not working on this. Unassign me. Thank you for understanding.

@Supriya2922
Copy link
Author

I'll work on this issue .Please assign me this.

@raysummee
Copy link
Collaborator

@Supriya2922 I have assigned you this issue. You may start working on it now.

@Tanuj-rai
Copy link

Can you please assign me this issue?

@raysummee
Copy link
Collaborator

@Supriya2922 I guess you're not working on this issue now. I'll assigned it to @Tanuj-rai .

@raysummee raysummee assigned Tanuj-rai and unassigned Supriya2922 May 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
assigned GSSoC22 Girl Script Summer of Code 2022 Level1
Projects
None yet
Development

No branches or pull requests

5 participants