-
Notifications
You must be signed in to change notification settings - Fork 344
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
Port threads
app templates to Django template syntax
#1152
Comments
@brylie I would like to work on this issue, can I give it a try? Btw I'm a beginner |
@sreejithdikru, sure. I'll assign you to the task. Please let us know if you have any difficulty with Backbone or Django templates. We can co-assign another person to the issue to assist. |
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as outdated.
This comment was marked as outdated.
I'd like to take over this task. |
@brylie, I will move I am planning to create |
Thanks that will help. 😃 I'm heading to bed now, but I'll check for more comments ASAP. |
Hello, @brylie. I see this issue is still open. Also, I have gone through the codebase, and the issues have not been fixed yet. Can you assign me, please? |
Hello, @brylie. I think the only task is deleting the api.py file in the threads app. All the other tasks seem to be completed. |
@gorkemarslan, can you confirm the status of this issue? |
I will take this @brylie. I will make the necessary changes where necessary |
hello @brylie , is this issue or any other issue that is beginner open source friendly up for grabs? i love this project!! |
We want to simplify the CiviWiki project by removing JavaScript SPA code in Backbone.js and using Django templates instead. We will work through the Backbone templates in the
threads
app to convert the template syntax from Backbone to Django. We want to maintain the visual styling. Below is a checklist of the relevant Backbone templates to be ported to complete this task.Note: Backbone markup is written between
{% verbatim %}
and{% endverbatim %}
template tags.Task
Port the following template from Backbone to Django view/template syntax.
convert API views to Django views (if necessary)
new_thread
get_thread
get_civi
get_civis
get_responses
create_civi
rate_civi
edit_civi
delete_civi
edit_thread
upload_civi_image
upload_thread_image
threads app shouldn't use Django REST framework
threads/api.py
filethreads app should contain only Class Based Views
all templates in
threads
app should{% for solution in solutions %}
all templates in
threads
app should not contain any{{# _.each(solutions, function (s) { }}
verbatim
tagsbackbone_templates
block contentThe text was updated successfully, but these errors were encountered: