Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Code cleanup: remove unused velocity templates #2481

Closed
muloem opened this issue Feb 13, 2018 · 1 comment
Closed

Code cleanup: remove unused velocity templates #2481

muloem opened this issue Feb 13, 2018 · 1 comment
Assignees
@muloem
Milestone
1.9.29 Cosmic Coc...

Comments

@muloem
Copy link
Member

muloem commented Feb 13, 2018

There are a number of velocity templates present in the repo that are not used anywhere or that are in parts of the code that are deprecated. We need to remove these along with the code that is not used anymore. Connected to the cleanup #555 and the the move to maven builds #2406
@muloem muloem self-assigned this Feb 13, 2018
muloem added a commit that referenced this issue Feb 13, 2018
@muloem
[#2481] Remove unused velocity templates and their references in the …
e2fff9b 
…code
muloem added a commit that referenced this issue Feb 13, 2018
@muloem
[#2481] Update copyright year and remove dependent method call
1266e30
valllllll2000 added a commit that referenced this issue Feb 13, 2018
@valllllll2000
Merge pull request #2482 from akvo/issue/2481-remove-velocity-templates
7d16504 
Remove unused velocity templates and their references (connect #2481)
@janagombitova janagombitova added this to the 1.9.29 C... C... milestone Feb 15, 2018
muloem added a commit that referenced this issue Feb 15, 2018
@muloem
[#2481] Remove ununsed velocity templates
658d46c
muloem added a commit that referenced this issue Feb 15, 2018
@muloem
[#2481] Remove unused instance creation velocity templates
b44c17d
stellanl added a commit that referenced this issue Feb 15, 2018
@stellanl
Merge pull request #2489 from akvo/issue/2481-remove-velocity-templat…
bb1d18d 
…es-2

Remove unused instance creation velocity templates (connect #2481)
muloem added a commit that referenced this issue Feb 15, 2018
@muloem
[#2481] Remove unused velocity template and its dependent methods
b84adfe
muloem added a commit that referenced this issue Feb 15, 2018
@muloem
[#2481] Remove unused velocity templates
e87e161
valllllll2000 added a commit that referenced this issue Feb 16, 2018
@valllllll2000
Merge pull request #2491 from akvo/issue/2481-remove-velocity-templat…
4d4ae04 
…es-4

Remove unused velocity templates (connect #2481)
valllllll2000 added a commit that referenced this issue Feb 22, 2018
@valllllll2000
Merge pull request #2487 from akvo/issue/2481-remove-velocity-templates
21b5daf 
Remove ununsed velocity templates (connect #2481)
@janagombitova
Copy link
Contributor

A lot has been merged for this issue. Is this part of the release? Anything I need to test? @muloem

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

@muloem muloem

Labels
None yet
Projects
None yet
Milestone
1.9.29 Cosmic Coconut
Development

No branches or pull requests
2 participants
@muloem @janagombitova