TheInvincible comments

Results 30 comments of


                                            TheInvincible

Timestamps in email notifications default to 24 hour time for new users

I am interested in this task, can I get a link the refers me to the exact code?

[Documentation] Write documentation for adding another language with i18n to the docs

I don't understand this issue, could you help with a little context?

Increase Unit Test Coverage to 50% Across all GoLang Files Throughtout the Project

/assign

Fix errcheck issue in web_test.go

> Thanks for the PR! > > There are still a couple of linter issues. Can you please take a look? > > ``` > Error: channels/web/web_test.go:84:17: Error return value...

Fix errcheck issue in web_test.go

I have made the corrections, @pvev and @hanzei. Hopefully the tests should pass now. Thank you!

Fix errcheck issue in web_test.go

/update-branch

Fix errcheck issue in web_test.go

Is there something I am yet to do, @hanzei?

Fix errcheck issue in web_test.go

I don't think the error is from this file. The MySQL error is referencing a nil pointer error in `handlers_test.go`.

Fix errcheck issue in web_test.go

Could not really find anything. Master ``` # github.com/mattermost/mattermost/server/v8/channels/store ..\store\store.go:328:57: undefined: model.ThreadMembershipForExport ..\store\store.go:513:39: undefined: model.MobileSessionMetadata # github.com/mattermost/mattermost/server/v8/config ..\..\config\client.go:28:76: c.ServiceSettings.EnableDesktopLandingPage undefined (type model.ServiceSettings has no field or method EnableDesktopLandingPage) ..\..\config\client.go:146:78: c.ExperimentalSettings.YoutubeReferrerPolicy...

Fix errcheck issue in web_test.go

> That seems like a compile error. What command did you use to run the tests? I used `go test -v`