-
Notifications
You must be signed in to change notification settings - Fork 5
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
Update 14.0.0.xml #7
Conversation
shorten 14 changelist
Some shortened suggestions: admin part:
user part:
Nevertheless I am not sure a user should be confronted with "2FA" and whether verification of people should be on the top. Would sort the second regular item topmost. Anyhow, decision on you. The idea behind those text is to BRIEFLY (yeah, i know it's hard) note down exciting things (regular users) and items that may require taking an action (admins. Btw, in this regard, dropped PHP 5 support would be noteworthy – or more positively phrased if you like). For everything else, there's the changelog (and announcement). |
No admin will care I bet. Because object store has to be setup before you install your Nextcloud. |
So for the admin part, what about:
First line inspired by nextcloud/server#7827 |
👍 those work for me, @blizzz |
oh, btw, after learning that
is experimental at best I am hesitant to include this, too. |
data/14.0.0.xml
Outdated
<item>Improved GDPR compliance capabilities with terms & conditions app and separate audit log</item> | ||
<item>Next generation of federated, cross-cloud sharing collaboration: Federation 2.0</item> | ||
<item>Swiftv3 auth support, logging to systemd, Kerberos auth to Samba & multi-IdP SAML</item> | ||
<item>Improved GDPR compliance with terms & conditions app; separate audit log</item> |
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.
the app is called terms of service
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.
thx!
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.
Ah, is being dropped anyway, cf #7 (comment)
data/14.0.0.xml
Outdated
<item>Verify the identity of a share recipient in a video call; and 2FA with Signal & Telegram</item> | ||
<item>Add a note to shares, see pending & deleted shares and search for files by content of comments</item> | ||
<item>Accessibility improvements including dark & high contrast themes and a dyslectic font</item> | ||
<item>Verify a share recipient in a video call; 2FA with Signal & Telegram</item> |
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.
If any, I would mention that 2FA via SMS is now supported. In contrast to Signal and Telegram, this has been tested more thoroughly. Still, the whole app must not be considered production ready. It's an early beta at best.
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.
On a general note, I always feel way more comfortable to only announce features that are done and stable.
Otherwise it leads to more pressure to the devs (and we all have plenty of things to do).
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.
@jospoortvliet that feedback is still there (and I still agree, but whatever i prefer a decision and see PD as owner of that).
Signed-off-by: Morris Jobke <[email protected]>
@blizzz I added a commit that uses the new formatting that is then also used by the script to re-generate the XML files. |
@MorrisJobke roger. lemme adjust em tests |
white spaces essentially
But this is not good, because then we need to update the tests on every transifex sync. |
<changelog href="https://nextcloud.com/changelog/#14-0-0"/> | ||
<whatsNew lang="en"> | ||
<regular> | ||
<item>Verify a share recipient in a video call; 2FA with Signal & Telegram</item> |
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 is still an issue, but I would merge it as it is now and then update it later on.
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.
I'll open a new issue for it
Because then the transifex bot can do it's job already ;) |
🤔 having working tests is a nice thing to do… perhaps checking instead for part of the content should be it
👍 |
shorten 14 changelist