Adding issues for TODOs

Signed-off-by: Yannick Bungers <git@innay.de>
This commit is contained in:
Yannick Bungers 2023-10-07 20:35:58 +02:00 committed by David Mehren
parent 7e1123e8a4
commit 8879b51344
16 changed files with 21 additions and 16 deletions

View file

@ -299,6 +299,7 @@ export default registerAs('authConfig', () => {
idpCert: process.env[`HD_AUTH_SAML_${samlName}_IDP_CERT`], idpCert: process.env[`HD_AUTH_SAML_${samlName}_IDP_CERT`],
clientCert: process.env[`HD_AUTH_SAML_${samlName}_CLIENT_CERT`], clientCert: process.env[`HD_AUTH_SAML_${samlName}_CLIENT_CERT`],
// ToDo: (default: config.serverURL) will be build on-the-fly in the config/index.js from domain, urlAddPort and urlPath. // ToDo: (default: config.serverURL) will be build on-the-fly in the config/index.js from domain, urlAddPort and urlPath.
// https://github.com/hedgedoc/hedgedoc/issues/5043
issuer: process.env[`HD_AUTH_SAML_${samlName}_ISSUER`], issuer: process.env[`HD_AUTH_SAML_${samlName}_ISSUER`],
identifierFormat: identifierFormat:
process.env[`HD_AUTH_SAML_${samlName}_IDENTIFIER_FORMAT`], process.env[`HD_AUTH_SAML_${samlName}_IDENTIFIER_FORMAT`],

View file

@ -106,7 +106,7 @@ export class IdentityService {
shouldSave = true; shouldSave = true;
} }
if (profilePicture) { if (profilePicture) {
// ToDo: sync image // ToDo: sync image (https://github.com/hedgedoc/hedgedoc/issues/5032)
} }
if (shouldSave) { if (shouldSave) {
identity.user = Promise.resolve(user); identity.user = Promise.resolve(user);

View file

@ -149,7 +149,7 @@ export class ConsoleLoggerService implements LoggerService {
day: '2-digit', day: '2-digit',
month: '2-digit', month: '2-digit',
}; };
//TODO make timestamp optional //TODO make timestamp optional (https://github.com/hedgedoc/hedgedoc/issues/5033)
const timestamp = new Date(Date.now()).toLocaleString( const timestamp = new Date(Date.now()).toLocaleString(
undefined, undefined,
localeStringOptions, localeStringOptions,

View file

@ -10,8 +10,7 @@ import { ServerStatusDto } from './server-status.dto';
@Injectable() @Injectable()
export class MonitoringService { export class MonitoringService {
// TODO Implement Server Status and other routes // TODO Implement Server Status and other routes (https://github.com/hedgedoc/hedgedoc/issues/478)
// (https://github.com/hedgedoc/hedgedoc/issues/478)
async getServerStatus(): Promise<ServerStatusDto> { async getServerStatus(): Promise<ServerStatusDto> {
return { return {
connectionSocketQueueLength: 0, connectionSocketQueueLength: 0,

View file

@ -67,6 +67,7 @@ export class WebsocketGateway implements OnGatewayConnection {
); );
if (notePermission < NotePermission.READ) { if (notePermission < NotePermission.READ) {
//TODO: [mrdrogdrog] inform client about reason of disconnect. //TODO: [mrdrogdrog] inform client about reason of disconnect.
// (https://github.com/hedgedoc/hedgedoc/issues/5034)
this.logger.log( this.logger.log(
`Access denied to note '${note.id}' for user '${username}'`, `Access denied to note '${note.id}' for user '${username}'`,
'handleConnection', 'handleConnection',

View file

@ -9,7 +9,9 @@ describe('Link gets replaced with embedding: ', () => {
cy.visitTestNote() cy.visitTestNote()
}) })
// TODO Add general testing of one-click-embedding component. The tests below just test a specific use of the component. // TODO Add general testing of one-click-embedding component.
// The tests below just test a specific use of the component.
// https://github.com/hedgedoc/hedgedoc/issues/5039
it('GitHub Gist', () => { it('GitHub Gist', () => {
cy.setCodemirrorContent('https://gist.github.com/schacon/1') cy.setCodemirrorContent('https://gist.github.com/schacon/1')

View file

@ -30,7 +30,6 @@ const EditorPage: NextPage<PageParams> = ({ params }) => {
/* /*
TODO: implement these in generateMetadata. We need these only in SSR. TODO: implement these in generateMetadata. We need these only in SSR.
See https://github.com/hedgedoc/hedgedoc/issues/4766 See https://github.com/hedgedoc/hedgedoc/issues/4766
But its problematic because we dont get the opengraph meta data via API. But its problematic because we dont get the opengraph meta data via API.

View file

@ -66,6 +66,7 @@ export const Sidebar: React.FC = () => {
onClick={toggleValue} onClick={toggleValue}
/> />
<ShareNoteSidebarEntry hide={selectionIsNotNone} /> <ShareNoteSidebarEntry hide={selectionIsNotNone} />
{/* TODO only show if user has permissions (Owner) (https://github.com/hedgedoc/hedgedoc/issues/5036) */}
<DeleteNoteSidebarEntry hide={selectionIsNotNone} /> <DeleteNoteSidebarEntry hide={selectionIsNotNone} />
<PinNoteSidebarEntry hide={selectionIsNotNone} /> <PinNoteSidebarEntry hide={selectionIsNotNone} />
</div> </div>

View file

@ -76,7 +76,7 @@ export const EntryMenu: React.FC<EntryMenuProps> = ({
<RemoveNoteEntryItem onConfirm={onRemoveFromHistory} noteTitle={title} /> <RemoveNoteEntryItem onConfirm={onRemoveFromHistory} noteTitle={title} />
{/* TODO Check permissions (ownership) before showing option for delete */} {/* TODO Check permissions (ownership) before showing option for delete (https://github.com/hedgedoc/hedgedoc/issues/5036)*/}
<ShowIf condition={userExists}> <ShowIf condition={userExists}>
<Dropdown.Divider /> <Dropdown.Divider />
<DeleteNoteItem onConfirm={onDeleteNote} noteTitle={title} /> <DeleteNoteItem onConfirm={onDeleteNote} noteTitle={title} />

View file

@ -65,6 +65,7 @@ export const ImportHistoryButton: React.FC = () => {
} }
//TODO: [mrdrogdrog] The following whole block can be shortened using our `readFile` util. //TODO: [mrdrogdrog] The following whole block can be shortened using our `readFile` util.
// But I won't do it right now because the whole components needs a make over and that's definitely out of scope for my current PR. // But I won't do it right now because the whole components needs a make over and that's definitely out of scope for my current PR.
// https://github.com/hedgedoc/hedgedoc/issues/5042
const fileReader = new FileReader() const fileReader = new FileReader()
fileReader.onload = (event) => { fileReader.onload = (event) => {
if (event.target && event.target.result) { if (event.target && event.target.result) {

View file

@ -22,7 +22,7 @@ const MAX_WIDTH_FOR_BUTTON_VISIBILITY = 1100
* *
* @param tocAst the {@link TocAst AST} that should be rendered. * @param tocAst the {@link TocAst AST} that should be rendered.
* @param width the width that should be used to determine if the button should be shown. * @param width the width that should be used to determine if the button should be shown.
* @param baseUrl the base url that will be used to generate the links //TODO: replace with consumer/provider * @param baseUrl the base url that will be used to generate the links //TODO: replace with consumer/provider (https://github.com/hedgedoc/hedgedoc/issues/5035)
*/ */
export const WidthBasedTableOfContents: React.FC<DocumentExternalTocProps> = ({ tocAst, width, baseUrl }) => { export const WidthBasedTableOfContents: React.FC<DocumentExternalTocProps> = ({ tocAst, width, baseUrl }) => {
if (width >= MAX_WIDTH_FOR_BUTTON_VISIBILITY) { if (width >= MAX_WIDTH_FOR_BUTTON_VISIBILITY) {

View file

@ -18,7 +18,8 @@ export class ForkAwesomeHtmlTagAppExtension extends AppExtension {
return [ return [
new SingleLineRegexLinter( new SingleLineRegexLinter(
forkAwesomeRegex, forkAwesomeRegex,
t('editor.linter.fork-awesome', { link: 'https://docs.hedgedoc.org' }) // ToDo: Add correct link here // ToDo: Add correct link here (https://github.com/hedgedoc/hedgedoc/issues/5021)
t('editor.linter.fork-awesome', { link: 'https://docs.hedgedoc.org' })
) )
] ]
} }