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

Obsidian Registration Entry Block Address Person Field Issues #5541

Closed
1 task done
frogydiak opened this issue Aug 9, 2023 · 7 comments
Closed
1 task done

Obsidian Registration Entry Block Address Person Field Issues #5541

frogydiak opened this issue Aug 9, 2023 · 7 comments

Comments

@frogydiak
Copy link

frogydiak commented Aug 9, 2023

Please go through all the tasks below

  • Check this box only after you have successfully completed both the above tasks

Please provide a brief description of the problem. Please do not forget to attach the relevant screenshots from your side.

  • The State in the Address field defaults to AZ.
  • The Address field will raise an error that Address Line 1 and City is required when it is not set to required.
    See screenshot below.

Expected Behavior

The address field State should be empty or defaults to the orgazanition state. The address field should not be required if it is not set as required.

Actual Behavior

image

Steps to Reproduce

  • At first I set the address field to required, then I got tired of entering the address everytime I change something in the form when testing so I unchecked the Required.

Rock Version

15.1

Client Culture Setting

en-US

@frogydiak frogydiak changed the title Obsidian Registration Entry Block Add Person Field Issues Obsidian Registration Entry Block Address Person Field Issues Aug 9, 2023
@sparkdevnetwork-service sparkdevnetwork-service added Status: In Progress Status: In Dev Queue This issue is being worked on, and has someone assigned. and removed Status: In Progress labels Sep 9, 2023
@joezimjs joezimjs added Fixed in v15.2 and removed Status: In Dev Queue This issue is being worked on, and has someone assigned. labels Sep 13, 2023
@joezimjs
Copy link
Contributor

Fixed with this commit: fb106eb

@JDShuman
Copy link

@joezimjs The address is still automatically pre-filling to AZ for us. We have consistently been updating to the latest public release, so we are now on v16.0. Am I missing a setting somewhere that sets our default state?

@joezimjs
Copy link
Contributor

@JDShuman Looking into this.

@joezimjs joezimjs reopened this Nov 17, 2023
@joezimjs
Copy link
Contributor

@JDShuman Under Admin Tools > General Settings > Global Attributes you'll find a setting for Organization Address

image

The default country and state are pulled from this address. I'm so sorry that I didn't address (no pun intended) this before marking it as fixed. Let me know if there's anything else I can do to help. If you don't need anything else, please re-close this issue. Thanks!

@JDShuman
Copy link

JDShuman commented Nov 17, 2023

Thanks @joezimjs. I wondered if that is where it was pulling from, but we've had that set for several years now.
Screenshot 2023-11-17 at 2 37 49 PM

Any ideas on why that wouldn't be pulling into our event registrations correctly? For additional context, we are running a web farm setup, but I've verified that the Global Attribute is populated on each server (as expected).

@nairdo
Copy link
Member

nairdo commented Nov 17, 2023

@JDShuman I'm afraid our fix had two parts to it and only one was in the v15.2 release. I just confirmed the bug is still in the v15.2 release but it is fixed in the upcoming v15.3 release (releasing next week) and v16.1 (via c6b8ce6 for issue #5607):

image

Sorry about that! We'll re-label this and correct the release note to be on the v15.3 release.

@joezimjs
Copy link
Contributor

@JDShuman Nick and I dug a little deeper and found out why I marked it as fixed but it was still causing issues for you. The change in v15.2 / v16.0 did fix this issue for Registration Attributes, but I didn't realize that the Person Field was actually using a different mechanism for gathering the default value. This is fixed in v15.3 / v16.1. I'm sorry for the confusion.

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants