Skip to content
This repository has been archived by the owner on Sep 16, 2022. It is now read-only.

Style Stipe pages #728

Open
vpetersson opened this issue Feb 19, 2020 · 13 comments
Open

Style Stipe pages #728

vpetersson opened this issue Feb 19, 2020 · 13 comments
Labels
story-points-unknown Unknown Story Points

Comments

@vpetersson
Copy link
Contributor

vpetersson commented Feb 19, 2020

We're not ready to roll out our Stripe integration yet, but I want to make sure that we've styled the required pages first. See this PR for details.

@dvidotti commented:
The issue is being addressed HERE

@rptrchv
Copy link
Contributor

rptrchv commented Feb 19, 2020

nodes

@vpetersson pls confirm everything looks ok. Unpaid nodes are struck out

@rptrchv
Copy link
Contributor

rptrchv commented Feb 19, 2020

register1

free plan ^

register2

standart plan ^

register3

standart plan (wrong card) ^

@rptrchv
Copy link
Contributor

rptrchv commented Feb 19, 2020

payment_fail

@vpetersson my current solution displays such errors on the separate page. we need to either style it or decide how to handle such errors different way.

@rptrchv
Copy link
Contributor

rptrchv commented Feb 19, 2020

Снимок экрана 2020-02-19 в 17 39 23

Снимок экрана 2020-02-19 в 17 39 52

Снимок экрана 2020-02-19 в 17 40 02

@vpetersson
Copy link
Contributor Author

@rptrchv I'll let @dvidotti take a stab at this before I comment.

@dvidotti
Copy link

At first, it seems to me that it would be better to have everything on just one page. As soon as I have a better understanding of the stripe branch I'll share my opinion on that.

@dvidotti
Copy link

@vpetersson this is the first design for the payment inside the register tab. Please share your thoughts on it, so I can know if it's the right way to go.

Captura de Tela 2020-02-20 às 13 02 53

Captura de Tela 2020-02-20 às 13 03 37

Captura de Tela 2020-02-20 às 13 03 49

Captura de Tela 2020-02-20 às 13 04 23

Captura de Tela 2020-02-20 às 13 04 32

@vpetersson
Copy link
Contributor Author

@dvidotti lots of comments here - but consider this round one :)

74932423-10ee7a00-53e2-11ea-8ab5-fdd81cd3482f

@dvidotti
Copy link

@vpetersson final of round one ;-)
I've made a new PR into the stripe branch. We have more detailed pics there:
#735

Captura de Tela 2020-02-22 às 17 49 16

Captura de Tela 2020-02-22 às 11 26 34

@vpetersson
Copy link
Contributor Author

@dvidotti much better. We still got that cut logo in the background tho.

@dvidotti
Copy link

@vpetersson I just pulled the server image a little outside the register box. The cut image was in the design project. It might be looking strange because we are missing the background-image here. Do you want to remove this image from this register pager?

@vpetersson
Copy link
Contributor Author

Yeah let's just remove it.

@dvidotti
Copy link

@vpetersson I removed the server image. The changes are in the PR #735

Captura de Tela 2020-02-29 às 18 29 00

@vpetersson vpetersson added the story-points-unknown Unknown Story Points label Mar 9, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
story-points-unknown Unknown Story Points
Projects
None yet
Development

No branches or pull requests

3 participants