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

bug(typo): change .value to .values in dictionary method call #68

Merged
merged 1 commit into from
Oct 3, 2022

Conversation

LUS24
Copy link
Contributor

@LUS24 LUS24 commented Sep 29, 2022

Changed value to values to solve the error AttributeError: 'dict' object has no attribute 'value'

Changed `value` to `values` to solve the error `AttributeError: 'dict' object has no attribute 'value'`
@vercel
Copy link

vercel bot commented Sep 29, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
rest-apis-flask-python ✅ Ready (Inspect) Visit Preview Sep 29, 2022 at 1:02PM (UTC)

@jslvtr
Copy link
Contributor

jslvtr commented Oct 3, 2022

Thank you @LUS24 ! Did you check if this typo is also on any of the lectures or other projects? 🤔

@LUS24
Copy link
Contributor Author

LUS24 commented Oct 3, 2022

Thank you @LUS24 ! Did you check if this typo is also on any of the lectures or other projects? 🤔

I haven't yet.

@LUS24
Copy link
Contributor Author

LUS24 commented Oct 3, 2022

Here I found two more ocurrences:
https://github.com/tecladocode/rest-apis-flask-python/search?q=items.value%28%29

I'll add them to the PR

@LUS24
Copy link
Contributor Author

LUS24 commented Oct 3, 2022

Added another PR to fix the other typo: #75

@jslvtr jslvtr changed the base branch from master to develop October 3, 2022 11:02
@jslvtr jslvtr changed the title Corrected typo bug(typo): change .value to .values in dictionary method call Oct 3, 2022
@jslvtr jslvtr merged commit e2afdca into tecladocode:develop Oct 3, 2022
@LUS24 LUS24 deleted the patch-3 branch October 20, 2022 09:56
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