Skip to content

Updating to current state of upstream#1

Open
MyrionPhoenixmoon wants to merge 1220 commits into
MyrionPhoenixmoon:masterfrom
isso-comments:master
Open

Updating to current state of upstream#1
MyrionPhoenixmoon wants to merge 1220 commits into
MyrionPhoenixmoon:masterfrom
isso-comments:master

Conversation

@MyrionPhoenixmoon
Copy link
Copy Markdown
Owner

No description provided.

ix5 and others added 26 commits June 13, 2022 11:11
[misc] Minor CSS nits, grey code bg, simplify JS i18n loading, Docker cache
This allows running the docker image with a non-root user (--user 2000)
with named/anonymous volumes (-v isso-db:/db).

When an empty volume is mounted into a container (non-bind mount), it is
prepopulated with the files at the mount path, including owners and
permissions. The issue is that since /db does not exist at docker build
time, the volume defaults to root ownership with 0755 permissions, which
prevents non-root users from creating the sqlite database. This change
explicitly creates the /config and /db directories with 1777 permissions
to allow any user to write to them.

Note that this has no impact on bind mounts, since these always use
permissions of the host filesystem.
Use the pretty/plain URL treewide.
Instead of hardcoding 25% width for each input block, create
dynamic-width blocks but keep a right-side margin of 1%
Margins have changed to be dynamic, necessitating an update
of the saved screenshots
This allows invocations of the image to change the CMD to
e.g. `isso.dispatch` for multi-site setups without having to
rebuild the image.
js: templates: Remove invalid `role="meta"` from comment template
Dockerfile: Explictly create volume paths and set permissions
js: i18n: fix translation fallback to English
[client] css: add auto word break
jelmer and others added 30 commits February 26, 2026 11:17
new language and plural config
galician (gl) translation
[Contribution] new language (gl)
Previously, the 'Log Out' button on the admin interface did
nothing when clicked in Firefox or Chrome.

Because 'admin-session' is created as a HostOnly cookie with no
domain specified, log_out() needs to refrain from attempting to
target the cookie using a domain as well.

To properly target the HostOnly cookie, the 'domain' attribute
should simply be omitted.

Refer to the login logic and cookie creation here:
- https://github.com/isso-comments/isso/blob/bd689143c3b9bd0fea83382bf4c1a1993586520c/isso/views/comments.py#L1439
docs: Update installation docs for newer versions of Fedora
Fix: Cookie deletion in Admin Interface
docs: multi-site-sub-uri: Add instructions for enabling multi-site support in Docker
Add missing entries: Arabic and Galician localisations, XSS fix,
Python version changes, pkg_resources migration, and comment
validation fix.
docs: multi-site-sub-uri: Clarify sub-URI setup and API endpoint detection
The guide claimed Mistune only understands three backticks, but it
actually supports 4-space indented code blocks too.

    import mistune
    md = mistune.create_markdown(escape=True)
    md('    indented code block\n    second line\n')
    md('```\nbacktick fenced\n```\n')

Both render as `<pre><code>...</code></pre>`.
docs: mistune: Remove incorrect claim about fenced code blocks
Add a configuration option `website-field` to control whether the Website input is shown in the comment form.
Default is true for backward compatibility.

Closes #916
js, config: Add option to show/hide website field in comment form
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.