agaric/agaric-coop
10
1
Fork 0
Code Issues 102 Pull requests Projects Releases Packages Wiki Activity

Review old content, in particular blog posts, for broken formatting and broken images #126

New issue
Closed
opened 2018-12-18 04:31:16 +00:00 by mlncn · 18 comments
mlncn commented 2018-12-18 04:31:16 +00:00 (Migrated from gitlab.com)
Copy link
  • Some code samples are being interpreted as HTML rather than viewed as code.
  • Some inline images are broken because (i think) they refer to image styles which are not on the new site (and/or that secure token craziness Drupal does now).

http://agaric-com.test/blog/marking-drupals-blog-posts-indieweb has examples of both but please find (and list here) and then perhaps fix others and assign this back to me to fix that one.

The solution to the code sample issue may be installing more text format handlers, but more recent blog posts seem to have imported OK so maybe old ones just need to be updated.

Work Remaining

  • Convert blog post titles to title case
  • Provide one sentence summary for all posts, remove any links from summary
  • Fix inline imagessnippets aren't displaying properly, despite use of tags.
  • Add relevant tag terms
* Some code samples are being interpreted as HTML rather than viewed as code. * Some inline images are broken because (i think) they refer to image styles which are not on the new site (and/or that secure token craziness Drupal does now). http://agaric-com.test/blog/marking-drupals-blog-posts-indieweb has examples of both but please find (and list here) and then perhaps fix others and assign this back to me to fix that one. The solution to the code sample issue may be installing more text format handlers, but more recent blog posts seem to have imported OK so maybe old ones just need to be updated. ## Work Remaining * [x] Convert blog post titles to title case * [x] Provide one sentence summary for all posts, remove any links from summary * [x] Fix inline imagessnippets aren't displaying properly, despite use of <code> tags. * [x] Add relevant tag terms
mlncn commented 2018-12-20 16:30:11 +00:00 (Migrated from gitlab.com)
Copy link

@FreeScholar can you take this?

@FreeScholar can you take this?
mlncn commented 2018-12-20 16:30:27 +00:00 (Migrated from gitlab.com)
Copy link

assigned to @FreeScholar

assigned to @FreeScholar
mlncn commented 2018-12-20 16:30:33 +00:00 (Migrated from gitlab.com)
Copy link

cc @cedewey

cc @cedewey
mlncn commented 2018-12-20 16:48:36 +00:00 (Migrated from gitlab.com)
Copy link

Related to #129 #125 #131 in the sense that they involve checking old content vs new.

Related to #129 #125 #131 in the sense that they involve checking old content vs new.
FreeScholar commented 2018-12-20 17:46:52 +00:00 (Migrated from gitlab.com)
Copy link

https://agaric-live-test.drutopia.org/blog?page=4
A p class=lead tag is appearing on the blog titled: Who are you in the drupal community" and I cannot copy paste the blog title in the list of blogs, the cursor grabs it like it is an image...

also happening on other listing pages:
https://agaric-live-test.drutopia.org/blog?page=2

https://agaric-live-test.drutopia.org/blog?page=4 A p class=lead tag is appearing on the blog titled: Who are you in the drupal community" and I cannot copy paste the blog title in the list of blogs, the cursor grabs it like it is an image... also happening on other listing pages: https://agaric-live-test.drutopia.org/blog?page=2
mlncn commented 2018-12-21 00:02:02 +00:00 (Migrated from gitlab.com)
Copy link

mentioned in issue #135

mentioned in issue #135
cedewey commented 2018-12-21 00:07:03 +00:00 (Migrated from gitlab.com)
Copy link

@FreeScholar I helped with blog post clean up today. I left off at https://agaric-live-test.drutopia.org/blog?page=4

Things to do for each post -

  1. Make post title use title case
  2. Limit summary to one sentence.
  3. Fix embedded images
  4. Fix code snippets (use <code> tag)
  5. Add/update terms in Tags field. One specific clean up item is anything that pertains to Drupal 7 should have a Drupal 7 tag instead of just Drupal.
@FreeScholar I helped with blog post clean up today. I left off at https://agaric-live-test.drutopia.org/blog?page=4 Things to do for each post - 1. Make post title use [title case](http://abeautifulbrand.com/capitalization-rules/) 2. Limit summary to one sentence. 3. Fix embedded images 4. Fix code snippets (use `<code>` tag) 5. Add/update terms in Tags field. One specific clean up item is anything that pertains to Drupal 7 should have a `Drupal 7` tag instead of just `Drupal`.
cedewey commented 2018-12-21 00:11:48 +00:00 (Migrated from gitlab.com)
Copy link

changed the description

changed the description
FreeScholar commented 2018-12-23 05:40:28 +00:00 (Migrated from gitlab.com)
Copy link

Reviewing

Reviewing
cedewey commented 2018-12-27 23:50:56 +00:00 (Migrated from gitlab.com)
Copy link

changed the description

changed the description
FreeScholar commented 2018-12-28 05:59:49 +00:00 (Migrated from gitlab.com)
Copy link

listing for these blog posts looks wacky in the listing-

Trip to GLADCamp 2014 in Los Angeles
listing: https://agaric-live-test.drutopia.org/blog?page=4

A brief review of the APC opcode cache
listing: https://agaric-live-test.drutopia.org/blog/brief-review-apc-opcode-cache

AHA -OK - it is a bug in that this error happens when there is a link in the Summary section - possibly any HTML might be bad too...
Some listings are inappropriately displaying the HTML like the p class=lead tag but if you save with FULL HTML setting, the p class=lead tags do not display on the page

Before I continue, let me know if I should fix manually @mlncn @cedewey

listing for these blog posts looks wacky in the listing- Trip to GLADCamp 2014 in Los Angeles listing: https://agaric-live-test.drutopia.org/blog?page=4 A brief review of the APC opcode cache listing: https://agaric-live-test.drutopia.org/blog/brief-review-apc-opcode-cache AHA -OK - it is a bug in that this error happens when there is a link in the Summary section - possibly any HTML might be bad too... Some listings are inappropriately displaying the HTML like the p class=lead tag but if you save with FULL HTML setting, the p class=lead tags do not display on the page Before I continue, let me know if I should fix manually @mlncn @cedewey
FreeScholar commented 2018-12-28 06:13:07 +00:00 (Migrated from gitlab.com)
Copy link

marked the checklist item Convert blog post titles to title case as completed

marked the checklist item **Convert blog post titles to title case** as completed
FreeScholar commented 2018-12-28 06:13:12 +00:00 (Migrated from gitlab.com)
Copy link

marked the checklist item Provide one sentence summary for all posts, remove any links from summary as completed

marked the checklist item **Provide one sentence summary for all posts, remove any links from summary** as completed
cedewey commented 2018-12-28 19:42:17 +00:00 (Migrated from gitlab.com)
Copy link

marked the checklist item Fix inline images as completed

marked the checklist item **Fix inline images** as completed
cedewey commented 2018-12-28 19:42:18 +00:00 (Migrated from gitlab.com)
Copy link

marked the checklist item Add relevant tag terms as completed

marked the checklist item **Add relevant tag terms** as completed
cedewey commented 2018-12-28 19:43:25 +00:00 (Migrated from gitlab.com)
Copy link

@FreeScholar yes, since we're already linking the whole teaser, we can't have links in the summary. I fixed the ones you pointed out and the last few blog posts. Thanks for your help!

@FreeScholar yes, since we're already linking the whole teaser, we can't have links in the summary. I fixed the ones you pointed out and the last few blog posts. Thanks for your help!
cedewey commented 2018-12-28 19:43:55 +00:00 (Migrated from gitlab.com)
Copy link

closed

closed
cedewey commented 2018-12-28 19:44:38 +00:00 (Migrated from gitlab.com)
Copy link

changed the description

changed the description
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
main
bbb-test
D9-upgrade
drush-9-aliases
provision-dewey
No results found.
Labels
Clear labels   
A11y

Accessibility-related issues

  
Automated Testing

   
Contributable

Issues with work that can (potentially) be contributed back to Drupal or other FLOSS projects.

  
Contributed

Tag issues where we have contributed work back to FLOSS projects (replace Contributable tag; the Contributed tag can be removed once the contribution is recorded in a blog or something.

  
Decision

An issue that requires a decision be made.

  
Design

Issues that should include a UX/design work or review.

  
Development

   
Drutopia

Issues related to Drutopia features or infrastructure, or are potential Drutopia enhancements (with tag Contributable).

  
Feature Request

   
IndieWeb

Anything related to interlinking web sites in a broadly IndieWeb fashion, see https://indieweb.org/

  
Infrastructure

   
Marketing

   
Needs documentation

When an issue has uncovered something that should be documented in a user guide or the project wiki or anywhere else outside the issue queue.

  
status::Abandoned

An issue that has been abandoned and will not be completed.

  
status::Blocked

An issue that is blocked for a reason outside Agaric's control.

  
status::Deploy

An issue that has been tested and is ready to be deployed.

  
status::Doing

   
status::Done

   
status::In Review

An issue that has been completed and is ready for a code and/or design review.

  
status::Needs Clarification

   
status::Test

An issue that has been completed and is ready to be tested.

  
status::To Do

An issue that is an agreed priority and has all information needed to be completed.

  
type::Bug

   
type::Task

   
type::User Story
No labels
A11y
Automated Testing
Contributable
Contributed
Decision
Design
Development
Drutopia
Feature Request
IndieWeb
Infrastructure
Marketing
Needs documentation
status::Abandoned
status::Blocked
status::Deploy
status::Doing
status::Done
status::In Review
status::Needs Clarification
status::Test
status::To Do
type::Bug
type::Task
type::User Story
Milestone
Clear milestone
No items
No milestone
Site Launch
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: agaric/agaric-coop#126
No description provided.
Delete branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?