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

Feature/access custom namespaces #64

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

hotdang-ca
Copy link

@hotdang-ca hotdang-ca commented Aug 11, 2022

I've noticed a bit of a trend -- new namespaces are introduced continually. And, this can continue.

But for now, I need support for:

  • namespaces not yet introduced (I'm parsing out g-core items... no idea what that is)
  • namespaces used incorrectly (my example code declares itself as an Atom feed, but does not follow the spec. I had to load it in as an RSS feed instead

This request adds support for reading in the raw text of unidentified tags (well... ALL tags at the moment. I'd love feedback regarding what to exclude).

This allows me to access:

...
<item>
  <g-core:price>2499.99</g-core:price>
</item>

via

feed.items!.first.customNamespace!.valueForKey('g-core:price')

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.

1 participant