Docs: Centralize Slack invitation URL - #17402
Conversation
Use the stable ASF redirect and share it through the MkDocs configuration to avoid repeated expiring invitation links. Generated-by: Codex
mxm
left a comment
mxm
left a comment
There was a problem hiding this comment.
Thanks Kevin! LGTM, just some minor remarks.
| flinkVersion: '2.1.0' | ||
| flinkVersionMajor: '2.1' | ||
| sparkVersionMajor: '4.1_2.13' | ||
| slackInviteUrl: &slackInviteUrl 'https://s.apache.org/iceberg-slack' |
There was a problem hiding this comment.
Who will be able to update the short URL? I think only the owner is able to do that, which makes the short URL a bit less appealing.
There was a problem hiding this comment.
agreed! i filed a ticket with ASF infra to see if we can add ldap group ownership to the url: https://issues.apache.org/jira/browse/INFRA-28184
if anything, we can change this link again via code (the current status quo 😄) but at least with this pr there are fewer places to change
There was a problem hiding this comment.
Sounds good. Thanks for following up with infra!
|
thanks for the reviews! merging this since it coalesce all the slack urls into 1 place. Im currently the only one that can modify the redirect for In any case we can create a new PR to override the |
3 participants
Context: #17398 (comment)
Use the stable ASF redirect and share it through the MkDocs configuration to avoid repeated expiring invitation links.
Maps
https://s.apache.org/iceberg-slack->https://join.slack.com/t/apache-iceberg/shared_invite/zt-4583kdp6p-ALpQGSa3O2tCYMtYwAMYNAFrom https://s.apache.org/s/private?action=list:
Rendered and verified locally:
(cd site && make serve-dev)Generated-by: Codex