search

drupalpune / campsite

0 stars 7 forks source link

Session node detailed page styling in-complete & remove unwanted fields #129

Closed AJV009 closed 1 year ago

AJV009 commented 1 year ago

Fix the theme for the detailed page of Session content type. image

Also remove the unwanted fields like Speaker Consent, Keynote Session, and add the Speaker and Co-Speaker name.

abhisekmazumdar commented 1 year ago

Thank you @AJV009 for creating the issue. We acknowledge these are necessary changes and we are working on them.

AJV009 commented 1 year ago

Awesome :zap:

Also, can we have more meaningful links? For people who are going to share on social media! image

A session/serialised-title instead of node/id maybe? @abhisekmazumdar Thanks :)

abhisekmazumdar commented 1 year ago

That's a good catch, for sure. we will update the allies.

abhisekmazumdar commented 1 year ago

Closing for now.

AJV009 commented 1 year ago

Let me know if I should help? I mean if the team is too busy I can look into too.

On Thu, 27 Jul 2023 at 19:43, Abhisek Mazumdar @.***> wrote:

That's a good catch, for sure. we will update the allies.

— Reply to this email directly, view it on GitHub https://github.com/drupalpune/campsite/issues/129#issuecomment-1653708610, or unsubscribe https://github.com/notifications/unsubscribe-auth/AKD7UAYYUZUIDBAOMWLN6P3XSJZQDANCNFSM6AAAAAA2T3ALJI . You are receiving this because you were mentioned.Message ID: @.***>

-- Master Alphons Jaimon, Explorer, Full Stack Drupal Engineer at QED42 https://ajv009.github.io +91-8237842347

abhisekmazumdar commented 1 year ago

The work is open to all. If you want, you can surely participate.

Ping me if you need any help with setting up on the Drupal Assoc India Slack.

AJV009 commented 1 year ago

hmm okay but there are two ways to take this right? Either set the aliases in content manually (easiest, just content change) or try writing some code to do all that automatically (BE development, the codebase is right here to get started, though I can't say how dedicated I am to work on the code right away :see_no_evil: )

abhisekmazumdar commented 1 year ago

We should use Pathauto. I can see Pathauto is already installed. I believe we just need to configure it for this specific content type, and we'll be all set. That will require a pull request (PR). I have created an issue for the same.

abhisekmazumdar commented 1 year ago

Don't worry, the team will take care of this.

AJV009 commented 1 year ago

Right :+1: Awesome thanks :)