Mentorship-android: Adding a better screen when the email confirmation link is pressed

Created on 25 Aug 2020  路  52Comments  路  Source: anitab-org/mentorship-android

Describe the bug
Currently, when the user fills his/her credentials in the signup page and presses the signup button, he/she gets a confirmation link on his/her registered email. When it is pressed, it leads us to a simple JSON response. Instead of that we can have a better-looking confirmation screen.

Expected behavior

  • Create the design for the confirmation screen
  • Implement it in code

Screenshots

Laptop description (please complete the following information):

  • Device: Lenovo 330
  • Version: Windows 10

Additional context
Provide a re-design of the email confirmation screen
TIP: You can download the apk from here and play around with the app, to understand what elements we want in this screen.

Kindly follow the AnitaB.org Branding Guidelines

Describe the solution you'd like
Submission format: png/jpeg / Figma URL and its implementation in code after pressing the email confirmation link
In the PR add 1 image of the design to /docs/UI folder.

Coding User Interface Open Source Hack Enhancement

Most helpful comment

I think as of now a simple JSON string is returned but you can create a simple HTML page or a component and put it in the root or app folder for a time being. Is that okay @anna4j @vj-codes ?

All 52 comments

@garimasingh128 I am willing to work on this issue.

@anna4j Can you suggest some design or ask the @anitab-org/design-team for some ideas for this?

@garimasingh128 you could checkout the redesign issues I've opened and include the branding guidelines in this issue. That'll defenitely help :)

If this issue is still open I would like to work on it-OSH

Can I work on this if it's available?

@Pranjal9999999 assigning you

Can I work on this if it's available?

Hey! Please look for another issue :)

I just updated the Additional content. @Pranjal9999999 Please check it out before progressing!

Yes I will take care of the description.Any other specific requirements, shall I add a background image.

Ya, you can be as creative as you want. Just follow the branding guidelines and try to be a little simple in design as the rest of the app is! :)

Can I give the solution in a png format.

Ya png/jpeg is the minimum requirement but you can also try a Figma URL or an Adobe XD link if you want to proceed further as it would be easier for us to make changes in the future.

20200902_225347_0000
Is it okay do you require any changes @garimasingh128

Looks good. I have a few suggestions:

  • You can add a text like "Kindly go to the app and login again" or something like this
  • Instead of having a square image in the centre, a blurred image in the whole background will be okay and the logo can be put at the side. But make sure it is clearly visible.

@anna4j @vj-codes You have any suggestions?

@Pranjal9999999 @anna4j @garimasingh128 I am not sure about using the word systerhood as the content was done when the organization was Systers and not Anitab.org.

@Pranjal9999999 please provide a web mode and phone mode comparative image too just to make sure it looks good on all sizes:)
And what is the source of image?

Ok I will instead put a blurred backdrop with the logo. Please tell me the exact text you want on the screen. I will provide you with the source.

Can you provide me the text that you want in the message @garimasingh128

Yeah! Sorry for the late response. I think it should say something like "Your email is confirmed. Welcome to AnitaB mentorship initiative. Please progress to the application to login." Of course, you can change words and take inspiration from confirmation mails we get normally.

Where do I add the path for the image, in which file.

@anna4j Can you tell? There's no docs/UI present currently.

Where do I add the path for the image, in which file.

You can add the png of image to docs/ui folder. You'll have to create that folder on your fork.
Also if you want to specify details you can write that in a .md file which will also be added to docs/ui folder

20200902_225347_0000
Is it okay do you require any changes @garimasingh128

One small change I completely didn't notice it before. It's "AnitaB.org Open Source" instead of "Systers"

i have taken this logo from the project itself

i have taken this logo from the project itself

Ya the project one is getting updated too. I guess you can leave this as it is now

Can you please review the pr.

Hey @Pranjal9999999 If you have created a PR then you can move on to the next issue. The pull requests would be reviewed over the course of time.

Yeah sure. Thanx!.

Hi is this issue available ?

Yeah. I think I can assign multiple assignees to this one. Let me know in case you are interested!

Yeah. I think I can assign multiple assignees to this one. Let me know in case you are interested!

Great i would love to work on this can you elaborate this issue a bit or correct me i have to code for this screen and update the png in docs folder right ? Also it should be responsible for both web and mobile version .

Yah *responsive for all screens. You have to do both the design part and coding the screen to link it to the confirmation screen. Take help from the previous comments. As it's mostly a design issue so multiple people can be assigned.

Yah *responsive for all screens. You have to do both the design part and coding the screen to link it to the confirmation screen. Take help from the previous comments. As it's mostly a design issue so multiple people can be assigned.

Yeah i'll go through them you can assigned it to me looking forward to work on this thanks

Can I also work on this @garimasingh128 ?

Okay @yash2189. 3 are fine for this issue. Not more than that. All the best!

@garimasingh128 @anna4j
I have created a mockup in Figma . How does this look for desktop and mobile?

desktop

phone

@anna4j thoughts? I actually like this one.

It's wonderful @yash2189

@yash2189 looks good , image source?

@yash2189 looks good , image source?

https://builtin.com/sites/default/files/styles/ckeditor_optimize/public/inline-images/women-in-tech-pillar-page-conferences-events-for-women_0.png

@vj-codes should I start with coding part? Also where should I add the template after coding.. Can you please tell me where template should be added?

@vj-codes should I start with coding part? Also where should I add the template after coding.. Can you please tell me where template should be added?

Sure go ahead

Yes thanks @anna4j @vj-codes it will approx >3 days as my final exams are near. Please do tell me where do I add the template.

I think as of now a simple JSON string is returned but you can create a simple HTML page or a component and put it in the root or app folder for a time being. Is that okay @anna4j @vj-codes ?

So is it fine if I keep the css and html file in the same folder @garimasingh128 ?

@garimasingh128 @vj-codes @anna4j I have completed the coding part,added responsiveness as well.
Please have a look at this. If it's correct then may I raise a PR for the same?

image

image

Yes please go ahead

So is it fine if I keep the css and html file in the same folder @garimasingh128 ?

For a time being, put all the files(assets, css, html etc) in a folder and keep it in app or root of the project. We will see how to integrate it later on.

@anna4j @garimasingh128 what should I write in the commit message since this is a coding and UI issue

Yeah mention it in the pull request description!

Should mention as fix or style @garimasingh128 ?

Please review my PR #912 @garimasingh128 @anna4j @vj-codes . Please tell me whether I need to change the location of email_verfication_template folder or is it correct?

@garimasingh128 @vj-codes @anna4j
Hey mentors please look into my PR thanks :)

Was this page helpful?
0 / 5 - 0 ratings

Related issues

annabauza picture annabauza  路  8Comments

isabelcosta picture isabelcosta  路  4Comments

iofall picture iofall  路  4Comments

isabelcosta picture isabelcosta  路  6Comments

epicadk picture epicadk  路  5Comments