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] website_event_track: ease searching tracks #162361

Conversation

spe-odoo
Copy link
Contributor

@spe-odoo spe-odoo commented Apr 18, 2024

Purpose

Simplify the process of finding tracks provided by a specific speaker.

Specifications

1> Add "Speaker" string to partner_id.
2> Add filter_domain so the record can be filtered out based on partner_id, speaker name, partner/contact email
and company name.
3> Reorder the fields to ease the search.
4> Field width increased for ease of input.

Task-3504040

@robodoo
Copy link
Contributor

robodoo commented Apr 18, 2024

@C3POdoo C3POdoo added the RD research & development, internal work label Apr 18, 2024
@spe-odoo spe-odoo force-pushed the master-website_event_track-imp-ease-tracks-spe branch from f8d8f15 to 3fe1dbb Compare May 14, 2024 07:12
@ibo-odoo ibo-odoo changed the title [IMP] website_event_track: ease search tracks by a person [IMP] website_event_track: ease search tracks May 14, 2024
@spe-odoo spe-odoo force-pushed the master-website_event_track-imp-ease-tracks-spe branch from 3fe1dbb to 9ce4a82 Compare May 14, 2024 09:08
@spe-odoo spe-odoo force-pushed the master-website_event_track-imp-ease-tracks-spe branch from 9ce4a82 to ddd8182 Compare May 28, 2024 06:07
Copy link
Contributor

@awa-odoo awa-odoo left a comment

Choose a reason for hiding this comment

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

Hello @spe-odoo

Left some comments on that one to try to polish a bit more.

Thanks & cheers!

@awa-odoo awa-odoo marked this pull request as ready for review June 4, 2024 13:53
@C3POdoo C3POdoo requested a review from a team June 4, 2024 14:05
@spe-odoo spe-odoo force-pushed the master-website_event_track-imp-ease-tracks-spe branch from ddd8182 to 42c864b Compare June 5, 2024 09:30
@spe-odoo spe-odoo changed the title [IMP] website_event_track: ease search tracks [IMP] website_event_track: ease searching tracks Jun 5, 2024
@spe-odoo
Copy link
Contributor Author

spe-odoo commented Jun 5, 2024

Hey @awa-odoo,

The changes are done. I also added the phone condition to the domain. Please have a look.

Thanks

@awa-odoo
Copy link
Contributor

awa-odoo commented Jun 5, 2024

Hello @spe-odoo

I think Luc was not a fan of searching based on phone number, so we can remove it to simplify.
Sorry for the confusion.

Cheers!

Purpose
=======
Simplify the process of finding tracks provided by a specific speaker.

Specifications
==============
1> Add "Speaker" string to partner_id.
2> Add filter_domain so the record can be filtered out
   based on partner_id, speaker name, partner/contact email
   and company name.
3> Reorder the fields to ease the search.
4> Field width increased for ease of input.

Task-3504040
@spe-odoo spe-odoo force-pushed the master-website_event_track-imp-ease-tracks-spe branch from 42c864b to c0b903d Compare June 5, 2024 12:07
@awa-odoo
Copy link
Contributor

awa-odoo commented Jun 5, 2024

@robodoo r+

robodoo pushed a commit that referenced this pull request Jun 5, 2024
Purpose
=======
Simplify the process of finding tracks provided by a specific speaker.

Specifications
==============
1> Add "Speaker" string to partner_id.
2> Add filter_domain so the record can be filtered out
   based on partner_id, speaker name, partner/contact email
   and company name.
3> Reorder the fields to ease the search.
4> Field width increased for ease of input.

Task-3504040

closes #162361

Signed-off-by: Warnon Aurélien (awa) <awa@odoo.com>
@robodoo robodoo added the 17.4 label Jun 5, 2024
@robodoo robodoo closed this Jun 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
17.4 RD research & development, internal work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants