You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
@brospars Thanks! In my limited experience, this hasn't been an issue, so that's why it's more conservative now. But that may be related to the particular courses, or that they've been smaller and we just haven't run into anyone changing their name yet. :)
If you want to submit a pull request changing this, please include links to EdX documentation and Discourse docs about this, as the plugin would have to make the same assumptions about which is the primary key and how changes are handled (eg, as in https://support.edx.org/hc/en-us/articles/115016004448-Can-I-change-my-edX-username- for EdX).
I'm also happy to brainstorm on how to verify this is works correctly for the various edge cases, since there's a bunch of risk in deploying that I likely won't be able to verify myself.
Activity
kevinrobinson commentedon May 7, 2019
@brospars Thanks! In my limited experience, this hasn't been an issue, so that's why it's more conservative now. But that may be related to the particular courses, or that they've been smaller and we just haven't run into anyone changing their name yet. :)
If you want to submit a pull request changing this, please include links to EdX documentation and Discourse docs about this, as the plugin would have to make the same assumptions about which is the primary key and how changes are handled (eg, as in https://support.edx.org/hc/en-us/articles/115016004448-Can-I-change-my-edX-username- for EdX).
brospars commentedon May 7, 2019
Yes Edx use the username as primary key, so this plugin could make the same assumption and if the email has changed then update it.
I'm leaving for some days but I will look into it next week
kevinrobinson commentedon May 7, 2019
@brospars Sounds good, pull requests welcome! 👍
I'm also happy to brainstorm on how to verify this is works correctly for the various edge cases, since there's a bunch of risk in deploying that I likely won't be able to verify myself.
brospars commentedon Jun 30, 2021
@kevinrobinson Hello just to let you know I've opened two PR would you mind taking a look ?
Merge pull request #2 from brospars/username-email-matching