Skip to content

Comments

Improving usability#41

Merged
LukeTowers merged 28 commits intowintercms:mainfrom
WebVPF:main
Jan 2, 2022
Merged

Improving usability#41
LukeTowers merged 28 commits intowintercms:mainfrom
WebVPF:main

Conversation

@WebVPF
Copy link
Contributor

@WebVPF WebVPF commented Dec 27, 2021

Improving usability - backend section

@bennothommo
Copy link
Member

@WebVPF great work with this. I've just left a couple of fixes for you to do.

WebVPF and others added 2 commits December 28, 2021 08:52
Co-authored-by: Ben Thomson <ben@abweb.com.au>
@WebVPF
Copy link
Contributor Author

WebVPF commented Dec 28, 2021

I will continue working when this pull request is accepted.

@WebVPF
Copy link
Contributor Author

WebVPF commented Dec 29, 2021

Remove none?
What to use where there is no language indicator? Indentation or three apostrophes?

@bennothommo
Copy link
Member

@WebVPF let's use three apostrophes as the standard from now on. In the case of none, simply change it so it's no language at all.

@WebVPF
Copy link
Contributor Author

WebVPF commented Dec 29, 2021

It is bad without none because it is not clear where the beginning of the code is.

WebVPF and others added 14 commits January 2, 2022 17:01
Co-authored-by: Luke Towers <github@luketowers.ca>
Co-authored-by: Luke Towers <github@luketowers.ca>
Co-authored-by: Luke Towers <github@luketowers.ca>
Co-authored-by: Luke Towers <github@luketowers.ca>
Co-authored-by: Luke Towers <github@luketowers.ca>
Co-authored-by: Luke Towers <github@luketowers.ca>
Co-authored-by: Luke Towers <github@luketowers.ca>
Co-authored-by: Luke Towers <github@luketowers.ca>
Co-authored-by: Luke Towers <github@luketowers.ca>
Co-authored-by: Luke Towers <github@luketowers.ca>
Co-authored-by: Luke Towers <github@luketowers.ca>
@LukeTowers LukeTowers merged commit e531648 into wintercms:main Jan 2, 2022
@LukeTowers
Copy link
Member

Thanks for all your work on this @WebVPF!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants