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.

Sign up for GitHub

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

Jump to bottom

Tehreem/fix config from yaml error #2582

Closed
wants to merge 52 commits into from
Closed

Tehreem/fix config from yaml error #2582

wants to merge 52 commits into from

Conversation

tehreem-sadat
Copy link

No description provided.

brittneyexline and others added 30 commits July 25, 2018 12:24
@brittneyexline
Revert "ENT-1052 Rename enterprise customer catalog uuid in querypara…
34b6c83 
…m to catalog"

This reverts commit 58174a6.

(cherry picked from commit 681cbb4)
@feanil
Update to the latest patch version of Django.
1fe5c3f 
(cherry picked from commit 325b8dd)
merger conflicts resolve master
eb15d94
@tehreem-sadat
add bluefin payment processor for credit card payment (using iframe)
bff8562 
add bluefin payment code

Fix double form validations

remove debugger

remove javascript alert messages

add requested changes: Use get for dict and remove string litterals

remove pep8 issues

remove pdb; remove configurations from oscar

remove post url from oscar

remove dummy values
Merge pull request #1 from ucsd-ets/tehreem/add_bluefin_payment_provider
5e100a9 
Add Bluefin Payment Processor
resolve errors after code merge
9e2529c
@tehreem-sadat
Merge pull request #2 from ucsd-ets/tehreem/payment_processor_resolve…
1743198 
…_errors

resolve errors after code merge
move configuration from ecommerce_env file to yaml file
58a15f6
@tehreem-sadat
Merge pull request #3 from ucsd-ets/tehreem/move_configuration_to_yaml
ab2abce 
move coonfiguration from ecommerce_env file to yaml file
show warning error on button click
91ab0f4
@tehreem-sadat
Merge pull request #4 from ucsd-ets/tehreem/resolve_refresh_transacti…
438f0ac 
…on_issue

show warning error on button click
@tehreem-sadat
add loadeing sign on payment button click
d628cfe
@tehreem-sadat
Merge pull request #5 from ucsd-ets/tehreem/display_loading_sign
abb164d 
Add loader on payment button click
@tehreem-sadat
Add issue credit and e2e code
f75a7ab
@tehreem-sadat
add env file
67ce597
@tehreem-sadat
Merge pull request #6 from ucsd-ets/tehreem/add_e2e_and_issue_credit_…
5ad9664 
…code

Add e2e and issue credit (refund) code
@tehreem-sadat
add strip code to remove ending /
065983b
@tehreem-sadat
initial chnages
8e1e531
@tehreem-sadat
delete popup method files
b75de21
@tehreem-sadat
add complete implementation code
7c3299c
@tehreem-sadat
refine code
d056d6c
@tehreem-sadat
refine code
b43033f
@tehreem-sadat
Merge pull request #7 from ucsd-ets/tehreem/change_e2e_add_stripe_line
393caa7 
change e2e => add strip line to run on dev server
@tehreem-sadat
changes done after dnaial review
6097000
@tehreem-sadat
use snake_case and remove * from imports
53c8dcf
@tehreem-sadat
code changes after review
c0a3782
@tehreem-sadat
Code to notify user after transaction
e6888b8
@tehreem-sadat
resolve response issues, disable client side processor and make upgra…
614ae1e 
…de changes
@tehreem-sadat
remove extra code
6867deb
@tehreem-sadat
improve checks
e969721
tehreem-sadat and others added 22 commits August 23, 2019 15:44
@tehreem-sadat
remove email backend and use base64 libraray for compatiblity
428c905
@tehreem-sadat
send email on rejected transaction
8436d6f
@tehreem-sadat
Add refund implementation code
66cd29e
@tehreem-sadat
Add detail docstings and change course purchase email signal
bffd056
@tehreem-sadat
handle return status to authorizenet using try/catch block
d29ccff
@tehreem-sadat
better doc-strings and handle eof
d8ddbcf
@tehreem-sadat
update cookie scenerio from courses list to one-time cookie
011c913
@tehreem-sadat
get authroizenet credentials from envion
435d69a
@tehreem-sadat
Merge pull request #8 from ucsd-ets/tehreem/authorize_payment_processor
3448130 
Integrate authorizenet payment processor
@tehreem-sadat
some changes found during uni tests
866a8d6
@tehreem-sadat
Merge pull request #9 from ucsd-ets/tehreem/authoriznet_fixes_found_d…
6935626 
…uring_tests

Add some changes found during uni tests
@tehreem-sadat @danialmalik
Add unit tests for AuthorizeNet payment processor
3063983
@tehreem-sadat
Merge pull request #10 from ucsd-ets/tehreem/authorize_payment_proces…
6221ea8 
…sor_tests

Add unit tests for AuthorizeNet payment processor
@tehreem-sadat
add cookie domain
2f62b44
@tehreem-sadat
Merge pull request #11 from ucsd-ets/tehreem/add_cookie_domain_name_h…
6b82c82 
…otfix

add cookie domain
@danialmalik
FIX: replace settings.get(KEY) with settings.KEY
299cc41
@tehreem-sadat
Merge pull request #12 from ucsd-ets/fix/settings-get
8c5fc53 
FIX: replace settings.get(KEY) with settings.KEY
@tehreem-sadat
change email backend for devserver
24dc839
@tehreem-sadat
Merge pull request #13 from ucsd-ets/tehreem/change_email_backend_for…
b2e8ecd 
…_dev_server

change email backend for devserver
@tehreem-sadat
change variable name
8d80b92
@tehreem-sadat
Merge pull request #14 from ucsd-ets/tehreem/change_variable_name
ce8acb0 
change variable name
@tehreem-sadat
fix config_form_yaml not found error
b11be61
@openedx-webhooks
Copy link

Thanks for the pull request, @tehreem-sadat! I've created OSPR-3842 to keep track of it in JIRA. JIRA is a place for product owners to prioritize feature reviews by the engineering development teams.

Feel free to add as much of the following information to the ticket:

  • supporting documentation
  • edx-code email threads
  • timeline information ("this must be merged by XX date", and why that is)
  • partner information ("this is a course on edx.org")
  • any other information that can help Product understand the context for the PR

All technical communication about the code itself will still be done via the GitHub pull request interface. As a reminder, our process documentation is here.

We can't start reviewing your pull request until you've submitted a signed contributor agreement or indicated your institutional affiliation. Please see the CONTRIBUTING file for more information.

@openedx-webhooks openedx-webhooks added needs triage open-source-contribution PR author is not from Axim or 2U labels Sep 11, 2019
@openedx-webhooks
Copy link

@tehreem-sadat Even though your pull request wasn’t merged, please take a moment to answer a two question survey so we can improve your experience in the future.

@danialmalik danialmalik deleted the tehreem/fix_config_from_yaml_error branch November 25, 2019 06:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
open-source-contribution PR author is not from Axim or 2U rejected
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@tehreem-sadat @openedx-webhooks @brittneyexline @feanil @danialmalik