Skip to content

[8.0][html_text] Excerpt generator. - #355

Merged
pedrobaeza merged 5 commits into
8.0from
unknown repository
Apr 4, 2016
Merged

[8.0][html_text] Excerpt generator.#355
pedrobaeza merged 5 commits into
8.0from
unknown repository

Conversation

@yajo

@yajo yajo commented Feb 11, 2016

Copy link
Copy Markdown
Member

This module adds a technical utility to allow the developer to get an excerpt from any HTML chunk.

You can choose the maximum amount of words or characters if you want.

See the README and inline docstrings for further details. Example usage found in OCA/website#163.

This module adds a technical utility to allow the developer to get an excerpt from any HTML chunk.

You can choose the maximum amount of words or characters if you want.

See the README and inline docstrings for further details.
Comment thread html_text/README.rst Outdated
Text from HTML field
====================

This module extracts text from any chunk of HTML, without HTML tags or

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This module extracts text from any chunk of HTML, without HTML tags or
attributes. You can chose either:

  • to truncate the result by amount of words or characters.
  • to append an ellipsis (or any character(s)) at the end of the result.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it's better to put "This module provides some technical modules that allow to extract ..."

Jairo Llopis added 3 commits February 12, 2016 09:58
Otherwise QWeb views would get broken badly.
Add a new `fail` parameter to allow exceptions to bubble up.

Control `ParserError` exceptions too.
@yajo

yajo commented Feb 17, 2016

Copy link
Copy Markdown
Member Author

Could somebody please tag this as "needs review"?

@pedrobaeza

Copy link
Copy Markdown
Member

Tagged as needs review

@pedrobaeza

Copy link
Copy Markdown
Member

Little comment, but 👍

@atchuthan

Copy link
Copy Markdown
Member

code reviewed 👍

@yajo

yajo commented Apr 4, 2016

Copy link
Copy Markdown
Member Author

Fixed.

@coveralls

Copy link
Copy Markdown

Coverage Status

Coverage increased (+1.2%) to 56.27% when pulling a0bedbe on grupoesoc:html_text into a249207 on OCA:8.0.

@pedrobaeza
pedrobaeza merged commit b8e5845 into OCA:8.0 Apr 4, 2016
@yajo
yajo deleted the html_text branch April 5, 2016 08:29
yajo pushed a commit to Tecnativa/website that referenced this pull request Jul 4, 2016
Add an excerpt and an image taken from the post to the blog summary page.

To make this work, these open PR need to be merged:

- OCA/server-tools#354
- OCA/server-tools#355
gfcapalbo pushed a commit to gfcapalbo/website that referenced this pull request Jul 19, 2016
Add an excerpt and an image taken from the post to the blog summary page.

To make this work, these open PR need to be merged:

- OCA/server-tools#354
- OCA/server-tools#355
gfcapalbo pushed a commit to gfcapalbo/website that referenced this pull request Sep 14, 2016
[8.0][website_blog_excerpt_img] New layout for blog.

Add an excerpt and an image taken from the post to the blog summary page.

To make this work, these open PR need to be merged:

- OCA/server-tools#354
- OCA/server-tools#355

Rephrasing.

Default value for `background_image` is usually something like `http://localhost/blog/our-news-1/post/blog-post-title-3?enable_editor=1`, not empty, so fixing here to avoid failure loading empty covers.

Simpler logic and better layout in phones.

Fix bug in generated images, add convenient classes.

The output of some images was a generator instead of an URL.

I add excerpt-txt and excerpt-img classes for make life easier for
themes.

Add basic styling.

All images now have the same height, and excerpts are 80 words long,
which mostly matches the image height.

Update for upstream module rename.

Initial commit

[IMP] Improved changing images and added backend view

[ADD] Readme

[FIX] New lines at end of xml files

[FIX] Removed unnecessary css class

[UPD] addons table in README.md

[ADD] setup.py

OCA Transbot updated translations from Transifex

first release of website_canonical_url

add a proper description

correct import according to review

FIX the review remarks

bug fixes + add languages

clean the code

fix README file

[FIX] remove empty section from README.rst

[IMP] add pagination support

[NEW][8.0] website_snippet_country_dropdown: New base module for country dropdown flags.

Add demo page.

CSS to Sass. Include compiled version and CSS map.

Fix typo.

Refactor snippet code.

- The `js_country_dropdown` class is added automatically by the template.
- Add `extra_class` variable to highest div element. You can set it to `has-error` for example.
- Add `default_value` variable to autopopulate the inputs.
- `default_country` is smarter now.

Translate to Spanish.

[UPD] addons table in README.md

[ADD] setup.py

[8.0][IMP][website_snippet_country_dropdown] Choose user's country by default.

With this patch, when the currently logged-in user has a country in its profile, it will be chosen by default for the flag.

[UPD] addons table in README.md

Update README.rst

Update README.rst

OCA Transbot updated translations from Transifex

Adding module website twitter no external dependencies

HB connection with token

working version with application token request

adding formatting
adding parametric support

images and frontend

Better data model
Parametric stuff

Major fix

views and js fixes

FIX Naming, now js is recognized

Human readable name
fix js error

Refactoring
Add theming
Theming in snippet
Menu
Documentation

Cleanup

remove required

Add dependency to website_parameterized_snippet

flake8

better placement of menues in knowledge

menu fix

fix menu permissions

added explicit security for therp website

admin access and user access

remove group, security managed on a model level

access to all

Feed lenght managment

remove therp branding

improve UL look and correct color of h5

flake8
gfcapalbo pushed a commit to gfcapalbo/website that referenced this pull request Sep 15, 2016
[8.0][website_blog_excerpt_img] New layout for blog.

Add an excerpt and an image taken from the post to the blog summary page.

To make this work, these open PR need to be merged:

- OCA/server-tools#354
- OCA/server-tools#355

Rephrasing.

Default value for `background_image` is usually something like `http://localhost/blog/our-news-1/post/blog-post-title-3?enable_editor=1`, not empty, so fixing here to avoid failure loading empty covers.

Simpler logic and better layout in phones.

Fix bug in generated images, add convenient classes.

The output of some images was a generator instead of an URL.

I add excerpt-txt and excerpt-img classes for make life easier for
themes.

Add basic styling.

All images now have the same height, and excerpts are 80 words long,
which mostly matches the image height.

Update for upstream module rename.

Initial commit

[IMP] Improved changing images and added backend view

[ADD] Readme

[FIX] New lines at end of xml files

[FIX] Removed unnecessary css class

[UPD] addons table in README.md

[ADD] setup.py

OCA Transbot updated translations from Transifex

first release of website_canonical_url

add a proper description

correct import according to review

FIX the review remarks

bug fixes + add languages

clean the code

fix README file

[FIX] remove empty section from README.rst

[IMP] add pagination support

[NEW][8.0] website_snippet_country_dropdown: New base module for country dropdown flags.

Add demo page.

CSS to Sass. Include compiled version and CSS map.

Fix typo.

Refactor snippet code.

- The `js_country_dropdown` class is added automatically by the template.
- Add `extra_class` variable to highest div element. You can set it to `has-error` for example.
- Add `default_value` variable to autopopulate the inputs.
- `default_country` is smarter now.

Translate to Spanish.

[UPD] addons table in README.md

[ADD] setup.py

[8.0][IMP][website_snippet_country_dropdown] Choose user's country by default.

With this patch, when the currently logged-in user has a country in its profile, it will be chosen by default for the flag.

[UPD] addons table in README.md

Update README.rst

Update README.rst

OCA Transbot updated translations from Transifex

Adding module website twitter no external dependencies

HB connection with token

working version with application token request

adding formatting
adding parametric support

images and frontend

Better data model
Parametric stuff

Major fix

views and js fixes

FIX Naming, now js is recognized

Human readable name
fix js error

Refactoring
Add theming
Theming in snippet
Menu
Documentation

Cleanup

remove required

Add dependency to website_parameterized_snippet

flake8

better placement of menues in knowledge

menu fix

fix menu permissions

added explicit security for therp website

admin access and user access

remove group, security managed on a model level

access to all

Feed lenght managment

remove therp branding

improve UL look and correct color of h5

flake8
yajo pushed a commit to Tecnativa/website that referenced this pull request Dec 19, 2016
Add an excerpt and an image taken from the post to the blog summary page.

To make this work, these open PR need to be merged:

- OCA/server-tools#354
- OCA/server-tools#355
yajo pushed a commit to Tecnativa/website that referenced this pull request Dec 22, 2016
Add an excerpt and an image taken from the post to the blog summary page.

To make this work, these open PR need to be merged:

- OCA/server-tools#354
- OCA/server-tools#355
yajo pushed a commit to Tecnativa/website that referenced this pull request Dec 22, 2016
Add an excerpt and an image taken from the post to the blog summary page.

To make this work, these open PR need to be merged:

- OCA/server-tools#354
- OCA/server-tools#355
yajo pushed a commit to Tecnativa/website that referenced this pull request Dec 27, 2016
Add an excerpt and an image taken from the post to the blog summary page.

To make this work, these open PR need to be merged:

- OCA/server-tools#354
- OCA/server-tools#355
yajo pushed a commit to Tecnativa/website that referenced this pull request Dec 28, 2016
Add an excerpt and an image taken from the post to the blog summary page.

To make this work, these open PR need to be merged:

- OCA/server-tools#354
- OCA/server-tools#355
yajo pushed a commit to Tecnativa/website that referenced this pull request Jan 4, 2017
Add an excerpt and an image taken from the post to the blog summary page.

To make this work, these open PR need to be merged:

- OCA/server-tools#354
- OCA/server-tools#355
rafaelbn pushed a commit to OCA/website that referenced this pull request Oct 12, 2017
* [8.0][website_blog_excerpt_img] New layout for blog.

Add an excerpt and an image taken from the post to the blog summary page.

To make this work, these open PR need to be merged:

- OCA/server-tools#354
- OCA/server-tools#355

* Default value for `background_image` is usually something like `http://localhost/blog/our-news-1/post/blog-post-title-3?enable_editor=1`, not empty, so fixing here to avoid failure loading empty covers.

* Simpler logic and better layout in phones.

Fix bug in generated images, add convenient classes.

The output of some images was a generator instead of an URL.

I add excerpt-txt and excerpt-img classes for make life easier for
themes.

Add basic styling.

All images now have the same height, and excerpts are 80 words long,
which mostly matches the image height.

Update for upstream module rename.

* OCA Transbot updated translations from Transifex

* [9.0][website_blog_excerpt_img] Migrate, LGPL.

Relicensed, migrated, new tests.
cristinamartinrod pushed a commit to Tecnativa/website that referenced this pull request Feb 15, 2019
* [8.0][website_blog_excerpt_img] New layout for blog.

Add an excerpt and an image taken from the post to the blog summary page.

To make this work, these open PR need to be merged:

- OCA/server-tools#354
- OCA/server-tools#355

* Default value for `background_image` is usually something like `http://localhost/blog/our-news-1/post/blog-post-title-3?enable_editor=1`, not empty, so fixing here to avoid failure loading empty covers.

* Simpler logic and better layout in phones.

Fix bug in generated images, add convenient classes.

The output of some images was a generator instead of an URL.

I add excerpt-txt and excerpt-img classes for make life easier for
themes.

Add basic styling.

All images now have the same height, and excerpts are 80 words long,
which mostly matches the image height.

Update for upstream module rename.

* OCA Transbot updated translations from Transifex

* [9.0][website_blog_excerpt_img] Migrate, LGPL.

Relicensed, migrated, new tests.
yajo added a commit to Tecnativa/website that referenced this pull request Apr 23, 2019
* [8.0][website_blog_excerpt_img] New layout for blog.

Add an excerpt and an image taken from the post to the blog summary page.

To make this work, these open PR need to be merged:

- OCA/server-tools#354
- OCA/server-tools#355

* Default value for `background_image` is usually something like `http://localhost/blog/our-news-1/post/blog-post-title-3?enable_editor=1`, not empty, so fixing here to avoid failure loading empty covers.

* Simpler logic and better layout in phones.

Fix bug in generated images, add convenient classes.

The output of some images was a generator instead of an URL.

I add excerpt-txt and excerpt-img classes for make life easier for
themes.

Add basic styling.

All images now have the same height, and excerpts are 80 words long,
which mostly matches the image height.

Update for upstream module rename.

* OCA Transbot updated translations from Transifex

* [9.0][website_blog_excerpt_img] Migrate, LGPL.

Relicensed, migrated, new tests.
SiesslPhillip pushed a commit to grueneerde/OCA-server-tools that referenced this pull request Nov 20, 2024
Syncing from upstream OCA/server-tools (11.0)
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.

5 participants