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

#164047056 Add CORS- origin #34

Merged
merged 1 commit into from
Apr 2, 2019
Merged

Conversation

Kibetchirchir
Copy link
Contributor

Description


A user can be able to consume the API without getting CROS origin error

Type of Change


  • New Feature

How has this been tested


  • Unit tests
  • End to end testing

Checklist


  • Tests are included to show/proof the feature works.

Pivotal Tracker


164047056

@Kibetchirchir Kibetchirchir temporarily deployed to aholympian-pr-34 March 30, 2019 11:02 Inactive
@Kibetchirchir Kibetchirchir temporarily deployed to aholympian-pr-34 March 30, 2019 11:09 Inactive
@Kibetchirchir Kibetchirchir changed the title fix(settings): Add CROS- origin #164047056 Add CROS- origin Mar 30, 2019
@JumaKahiga
Copy link
Collaborator

Thanks for working on this issue Chirchir.

@Kibetchirchir Kibetchirchir temporarily deployed to aholympian-pr-34 April 2, 2019 06:05 Inactive
@Kibetchirchir Kibetchirchir changed the title #164047056 Add CROS- origin #164047056 Add CORS- origin Apr 2, 2019
requirements.txt Outdated
@@ -37,6 +40,8 @@ MarkupSafe==1.1.1
mccabe==0.6.1
mock==2.0.0
more-itertools==6.0.0
mypy==0.670
Copy link
Contributor

@JoanNgatia JoanNgatia Apr 2, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we need the libraries mypy, mypy-extensions, typed-ast and typing-extensions for this feature fix?
If they didn't get added by installing some of the other libraries, then we can remove them.

- Add CORS origin

[Fixes #164047075]
@JoanNgatia JoanNgatia merged commit e0231ab into develop Apr 2, 2019
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.

None yet

3 participants