-
Notifications
You must be signed in to change notification settings - Fork 7
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
Add user alerts where no ticket type is configured #18
Comments
Not sure I understand. You can only link a Ticket Type (that's step 2) after you link the Event (that's step 1), but you suggest preventing them from doing step 1 unless they've done step 2. I'm probably misreading your comment. Can you explain further? |
@twomice unless you link a Ticket Type, syncing does not work. I completed step 1, tried to sync, did not work. After I went back and added a ticket type in the matching Civi Event, and completed step 2, syncing worked. So, at least add a warning/alert in the UI, after step 1, that step 2 must be completed for the sync to work. |
Okay, that would work. I've adjusted the title, added "enhancement" label, and inserted a ticket summary to this effect at the top of the ticket. I don't have time to make this change at the moment, but PR is definitely welcome. |
So maybe a PR with a basic HOWTO doc, which I can make a start on.... |
FYI, I've updated the README file with a note on project status. Because we don't use EB here, it's hard for me to keep this up to date, though I'm hopeful some EB user organization will be interested in sponsoring the work to get it working agian. NOTICE of project statusThis project is known to have significant issues with the current EventBrite API.Although it worked well when first released, EB has made changes to their API, and this extension is very likely not to work completely, or perhaps in any usable way. If you're a current EventBrite user who'd like to help sponsor the work needed get this working again, please contact us directly via https://joineryhq.com/ to discuss your needs. |
Ticket Summary (by @twomice):
Per @joemcl's comment, the task at hand is to improve user experience by alerting user when a linked event has no linked ticket types, which are required for any data to be transferred.
Original starting comment from @joemcl :
Apparently a linked Ticket Type is required in the settings - which makes sense, and I had to create a ticket type in the Eventbrite event - but that wasn't obvious until I tried to figure out why attendees were not syncing. Once I selected the ticket type, then the attendees were pulled in when running the Scheduled Job manually. So, a suggestion - don't let users save a new Event in Settings unless a Ticket Type is linked :) .
The text was updated successfully, but these errors were encountered: