-
-
Notifications
You must be signed in to change notification settings - Fork 11
Added OM descriptor #58
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
Open
alethkit
wants to merge
53
commits into
master
Choose a base branch
from
om-description
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Also make some stylistic italic tags into <i>. <em> is for emphasis.
The website is canonically live at https://www.srcf.net, but for alternate copies of the site (e.g. dev versions), it's a pain to have URLs here and there hardcoded with the live website's domain that take the browser to the live site from a dev copy without much warning. Most of these cross-site links seem to have emerged from pages that were originally authored in Markdown elsewhere (e.g. on notes.srcf.net) and then copied into the site source as HTML without adapting the URLs. Links are expected to stay within the same copy of the site wherever possible (or to point at counterparts, such as a dev version of Control Panel if navigating from a dev version of the main site). Fix those links, so that they work as expected. Note that there are a few special exceptions of services that live under the main site's domain for historical reasons but are logically separate from it, such as the web terminal at /terminal/ and phpMyAdmin/phpPgAdmin at /php{my,pg}admin/. These are left alone, partly because they are not really anything to do with the main site itself and also because dev versions of the site do not offer those services, given that they're 'mounted' under https://www.srcf.net/ by special server config. We often use SSI variables to substitute the actual site base URL in hyperlinks, which also allows for a copy of the site to be hosted at not-the-root (e.g. with a base URL of 'https://example.com/srcf-web/') although I've chosen to use simpler relative URLs where I can, and absolute URLs where it makes sense -- namely, when accessing the '/_srcf/' meta-directory that hosts stylesheets and other assets. Also, fix the homepage donation button that was nonfunctional.
Previously, it would accidentally block everything
The following files were ignored in the conversion as they are not "code" and will not need to be edited for maintenance reasons: - minutes/*.txt - images/*.svg
* so that it displays correctly in <title> tags * the use of en dashes in those contexts is incorrect
Fix my name :)
Added notes taken during the meeting, need to add additional notes from impromptu group chat made to schedule meeting.
A bunch of relevant events were discussed in a Whatsapp group chat (now closed) and partly in #executive, which are related to this meeting, so I have appended the events here. Typo in the 25th anniversary meeting section: s/now/know
Accidentally removed the header and footer in the last commit. Thanks go to dcc52 for pointing this out.
Markdown did not successfully convert the list to an UL list. Thanks go to dcc52 for spotting it.
December meeting minutes have now been added!
Don't imply that SRCF membership ends when leaving the university
Unclear what the non-labelled minutes were supposed to be. Turns out that they were for ordinary meetings, but this was not clearly marked. This patch rectifies this.
CHTJonas
requested changes
Oct 17, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This PR isn't correct. The non-labelled rows are minutes from committee meetings, not ordinary meetings.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Unclear what the non-labelled minutes were supposed to be. Turns out that they were for ordinary meetings, but this was not clearly marked.
This patch rectifies this.