#133 [Docs] Add to FAQ the solution to no messages in Archives
Merged 5 years ago by jflory7. Opened 5 years ago by shraddhaag.
fedora-commops/ shraddhaag/fedora-happiness-packets modify-setup-development-docs  into  master

file modified
+24 -2
@@ -6,13 +6,35 @@ 

  It includes troubleshooting steps and other project details.

  

  

- ERROR: Couldn't connect to Docker deamon at http+docker://localhost-is it running? 

+ ERROR: Couldn't connect to Docker deamon at http+docker://localhost-is it running?

      Verify the logged in user is member of the docker group.

      To verify logged in user run::

  

   	sudo usermod -aG docker ${USER}

-     

+ 

      If logged in user is not member of the docker group add it using::

  

  	sudo gpasswd -a${USER}

  

+ No sample messages on the main page and no messages in Archives, even though both sender and receiver approved to display message publicily.

+ 

+     This is due to ``admin_approved_public`` not being set. This has to be done manually using shell.

+     Steps to resolve:

+ 

+     1. Access the shell of the container web using::

+ 

+         docker-compose exec web sh

+ 

+     2. Acess the Django shell using::

+ 

+         python manage.py shell

+ 

+     3. Import the ``Message`` model and query the necessary message.

+        [Assuming the variable ``message`` points to the queried object]

+ 

+     4. Set the ``admin_approved_public`` attribute to True and save the modified object::

+ 

+         message.admin_approved_public = True

+         message.save()

+ 

+     Now when you access Archives, the message can be seen.

Q - What is a summary of your change?
A - This commit adds the steps to follow when no sample messages on the main page and no
messages in Archives, question is raised. The necessary steps are added in the faq.html template.

Q - Why is this change helpful?
A - While navigating the site, devs working on the project wonder why after creating test packages, none appear in Archives. This helps answer that query.

Note: The changes successfully passed the test-docs.sh.

Metadata Update from @jflory7:
- Pull-request tagged with: improvement, type - docs, type - summer coding
- Request assigned

5 years ago

Hi @shraddhaag, thanks for the PR. This PR has a merge conflict from #132. Could you rebase your changes on top of what is currently in master branch?

Git conflicts are notoriously tricky if you haven't tried fixing one before. If you need any resources or guidance on how to do this, let me know and I can share some links to help you resolve the merge conflict.

Metadata Update from @jflory7:
- Pull-request tagged with: needs changes

5 years ago

rebased onto 7d4f1e76e4cf82e62c4150858daf7b0bc9917034

5 years ago

Hi @jflory7, I rebased the changes on the current master branch. I completely agree, Git conflicts are notoriously tricky!

A nitpick, but the asterisk for a list is not needed. See the previous question as an example.

To make git diffs easier to review in the future, could you keep each full sentence to its own line? In this case, merge line 6 into 5.

@shraddhaag Thanks, this is perfect. I see now this is the answer to the question I just asked you in #69. :stuck_out_tongue:

I left two in-line comments with nitpick edits after I previewed the PR locally. Please address the feedback, squash/rebase your commits together, and we will merge the PR. :smile:

rebased onto f7392bc

5 years ago

@jflory7 I rebased it, could you please check if any further changes are needed? :)

@shraddhaag Excellent! Looks great. Merging. :clapper:

Metadata Update from @jflory7:
- Pull-request untagged with: needs changes

5 years ago

Pull-Request has been merged by jflory7

5 years ago