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.

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

[IMP] theme_*: adapt s_split_intro snippet #876

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

chgo-odoo
Copy link
Contributor

@chgo-odoo chgo-odoo commented Aug 21, 2024

This PR reviews the occurrences of s_split_intro snippet across design themes.

task-4077616
part of task-4077427

@robodoo
Copy link
Collaborator

robodoo commented Aug 21, 2024

Pull request status dashboard

@chgo-odoo chgo-odoo force-pushed the master-new-snippets-connections-chgo branch 2 times, most recently from c8027ee to 36aa034 Compare August 21, 2024 11:02
@chgo-odoo chgo-odoo force-pushed the master-new-snippets-connections-chgo branch 4 times, most recently from f16ab9e to 7f97135 Compare August 27, 2024 18:54
@chgo-odoo chgo-odoo changed the title [IMP] theme_*: adapt s_connections snippet [IMP] theme_*: adapt s_split_intro snippet Aug 27, 2024
Copy link
Contributor

@anso-odoo anso-odoo left a comment

Choose a reason for hiding this comment

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

Hey @chgo-odoo some comments for you

I noticed that the images you are using are very pixelised, can we do something to fix that ?

theme_beauty/views/images.xml Outdated Show resolved Hide resolved
theme_aviato/views/images_library.xml Outdated Show resolved Hide resolved
@chgo-odoo chgo-odoo force-pushed the master-new-snippets-connections-chgo branch 3 times, most recently from aafd3f3 to dbde21f Compare September 11, 2024 14:21
Copy link
Contributor

@anso-odoo anso-odoo left a comment

Choose a reason for hiding this comment

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

Hey @chgo-odoo, some record urls need to be fixed 👍

theme_avantgarde/views/images_library.xml Outdated Show resolved Hide resolved
theme_avantgarde/views/images_library.xml Outdated Show resolved Hide resolved
theme_beauty/views/images.xml Outdated Show resolved Hide resolved
@chgo-odoo chgo-odoo force-pushed the master-new-snippets-connections-chgo branch from dbde21f to 363f220 Compare September 19, 2024 07:06
@chgo-odoo chgo-odoo force-pushed the master-new-snippets-connections-chgo branch 2 times, most recently from 2eabc14 to 5602f6c Compare October 7, 2024 08:19
*: anelusia, artists, avantgarde, aviato, beauty, bewise, bistro,
bookstore, buzzy, cobalt, enark, kiddo, loftspace, monglia, nano, notes,
odoo_experts, orchid, real_estate, treehouse, vehicle, yes, zap.

This commit reviews the occurrences of `s_split_intro` snippet across
design themes.

task-4077616
part of task-4077427
@chgo-odoo chgo-odoo force-pushed the master-new-snippets-connections-chgo branch from 5602f6c to 7dfe2cc Compare October 7, 2024 13:53
@chgo-odoo chgo-odoo marked this pull request as ready for review October 8, 2024 08:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants