Skip to content

Conversation

@Abhiishekk31
Copy link
Contributor

Worked on issue #783

@jmilljr24
Copy link
Collaborator

@Abhiishekk31 Looks like you have some commits from your last PR. Lets start this over to make it clean. Can you try following this guide. The goal being to get your main branch update to date with our main branch. Then create a new working branch for this issue. Make the changes, commit, push and open a new PR.

Also its a bit easier if you put the screenshots in with the PR instead of the issue page.

I'm thinking the new text will work better on mobile if we put it above the links instead of on the side. Maybe something like this?

    <% if event.event_registrations.exists?(registrant_id: current_user.id) %>
      <div class="flex flex-col items-center mt-2">
        <p class="font-md text-sm text-gray-500">Add to Your Calendar</p>

        <div class="flex gap-2 items-center justify-center w-full text-sm text-gray-700">
          <%= event.calendar_links %>
        </div>
      </div>
    <% end %>

@jmilljr24
Copy link
Collaborator

#813

@jmilljr24 jmilljr24 closed this Jan 30, 2026
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.

2 participants