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.

Sign up for GitHub

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

Jump to bottom

Admin defined profile fields not back-compatible #7634

Closed
beck24 opened this issue Dec 6, 2014 · 4 comments
Closed

Admin defined profile fields not back-compatible #7634

beck24 opened this issue Dec 6, 2014 · 4 comments
Assignees
@beck24
Labels
bc break easy

Comments

@beck24
Copy link
Member

beck24 commented Dec 6, 2014

At some point the field indexes started counting from 0, now they count from 1
So any older sites (like the elgg community) that had admin defined fields - the first fields are lost and unrecoverable if edited.
@beck24 beck24 mentioned this issue Dec 6, 2014
Merged
@beck24
Copy link
Member Author

beck24 commented Dec 6, 2014

@Srokap - do you have a script adding the in progress label and assigning people or are you just really on the ball?

@beck24
Copy link
Member Author

beck24 commented Dec 22, 2014

Fixed in #7635

@beck24 beck24 closed this as completed Dec 22, 2014
@Srokap
Copy link
Contributor

Srokap commented Dec 24, 2014

Labels are coming from http://waffle.io/elgg/elgg

@beck24
Copy link
Member Author

beck24 commented Dec 24, 2014

Oh, good to know thanks.
note: I deleted the comment produced by the bounced email notification

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@beck24 beck24

Labels
bc break easy
Milestone
No milestone
Development

No branches or pull requests
2 participants
@beck24 @Srokap