URL construction for member list access using base URL and org ID#467
URL construction for member list access using base URL and org ID#467anagha-cmd wants to merge 3 commits intoCSSUoB:mainfrom
Conversation
|
Using the base URL in this way is a good idea! One issue I could forsee is if an alternative membership system didn't have a concept of organisation IDs. |
MattyTheHacker
left a comment
There was a problem hiding this comment.
The base url stuff here isn't necessary as it's unlikely that any organisation other than the guild will have the same url format so that can be removed. If we're looking at supporting other SUs I suspect that'll be a larger issue with significant refactoring needed to support the different member list formats
See #253 |
|
Yeah I think until we're ready to start actioning the config stuff we assume we're just working with the guild. We can start implementing compatibility with other SUs when that's done |
I'd say we can be even more generic than that and just say "membership systems"! /lh |
Signed-off-by: Matt Norton <matt@carrotmanmatt.com>
|
@anagha-cmd are you still working on this? if not I'll go ahead and close the pr and re-assign the issue |
|
@MattyTheHacker hey sorry, was stuck in some other work. Please go ahead and re-assign this issue |
This PR modifies how member list URLs are constructed by using a configurable base URL from environment variables instead of hardcoded values. This change improves flexibility and allows the bot to work with different membership systems. Closes #455
Changes made:
MEMBERS_LIST_BASE_URLas a required environment variableconfig.pycogs/make_member.pyto use the base URL from settingsREADME.mdand.env.exampleto reflect these changesTesting: