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

Render links for XLSForm error URLs #720

Open
matthew-white opened this issue Jan 26, 2023 · 0 comments
Open

Render links for XLSForm error URLs #720

matthew-white opened this issue Jan 26, 2023 · 0 comments
Labels
enhancement New feature or behavior help wanted

Comments

@matthew-white
Copy link
Member

Now that we have resolved #687, Frontend will render a link for an XLSForm warning ending with "Learn more: [some URL]". It would be great if we could do the same thing for an XLSForm error. For example:

Screenshot

It's not easy to take the same approach here as we did for #687 because right now, the FormNew component doesn't itself render the error. Instead, this.request() in FormNew shows the alert by using the alert object in the container, which causes the Alert component in the Modal component to be shown. The only role that FormNew plays is extracting the error message from the details property of the Problem response and prefixing it with the string "The XLSForm could not be converted:". It does that using the problemToAlert option passed to this.request().

@matthew-white matthew-white added enhancement New feature or behavior help wanted labels Jan 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or behavior help wanted
Projects
None yet
Development

No branches or pull requests

1 participant