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

[Signup fields III] Profile form #14966

Merged
merged 15 commits into from
Jul 25, 2019
Merged

[Signup fields III] Profile form #14966

merged 15 commits into from
Jul 25, 2019

Conversation

gonzaloriestra
Copy link
Contributor

@gonzaloriestra gonzaloriestra commented Jun 12, 2019

Related to CartoDB/product#337

Added the new fields to the user profile and enabled synchronization with central.

⚠️ DO NOT MERGE UNTIL #14963 IS DEPLOYED

@gonzaloriestra
Copy link
Contributor Author

retest this please

@gonzaloriestra gonzaloriestra changed the title Add signup fields to user forms [Signup fields IV] Profile form Jun 13, 2019
@gonzaloriestra gonzaloriestra changed the title [Signup fields IV] Profile form [Signup fields III] Profile form Jun 13, 2019
@gonzaloriestra
Copy link
Contributor Author

retest this please

@simon-contreras-deel
Copy link
Contributor

simon-contreras-deel commented Jun 14, 2019

Acceptance 💚 ❓

Everything seems to be ok, in cartodb, central and dbs, but in all cases, I have received a 504 error (gateway timeout) by nginx in the account creation. I suppose, this is happening in master too and it is not due to these changes.

@simon-contreras-deel
Copy link
Contributor

simon-contreras-deel commented Jun 14, 2019

It is happening (504 error) with master branchs too. So I would say:

Acceptance 💚

@simon-contreras-deel simon-contreras-deel removed their assignment Jun 17, 2019
@alejandraarri alejandraarri merged commit d53af14 into master Jul 25, 2019
@alejandraarri alejandraarri deleted the signup-fields-front branch July 25, 2019 10:59
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

5 participants