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

Create Course Certificate #20

Merged
merged 49 commits into from
Aug 1, 2023
Merged

Create Course Certificate #20

merged 49 commits into from
Aug 1, 2023

Conversation

nickdelgrosso
Copy link
Contributor

No description provided.

@nickdelgrosso nickdelgrosso linked an issue Jul 25, 2023 that may be closed by this pull request
Make Course Certificates for a Given Workshop #18
Closed
4 tasks
@nickdelgrosso nickdelgrosso marked this pull request as draft July 25, 2023 09:56
@nickdelgrosso
added start and end times
6fc9f66
@nickdelgrosso
added session ids
3067e95
@nickdelgrosso
started get_session_details
0b602b3
@nickdelgrosso
added session planned start and planned end
66000aa
@nickdelgrosso
updated to names around use case of planning a workshop
a74332e
@nickdelgrosso
bugfix
26a0d1d
@nickdelgrosso
remove hardcoding by adding some property testing
3fdb376
@nickdelgrosso
added full workshop workflow
ffd4811
@nickdelgrosso
Clearer split between worflow and repo needs, added Data Transfer Obj…
818f605 
…ect interface
@nickdelgrosso
added property test of length of get_workshops
b747ad8
@nickdelgrosso
R: minor
3a15954
@nickdelgrosso
clearer dto-> worfklow output adjustment
c862a26
@nickdelgrosso
split code into multiple submodules to help improve readability, make…
b99f7d8 
… architecture a bit clearer
@nickdelgrosso
zoom repo, workshop getter
548b0ae
@nickdelgrosso
zoom repo, added session getter
a4601b4
@nickdelgrosso
fix mock reuquester to work on the json() method of the response object.
db45bae
@nickdelgrosso
almost implemented list_meetings
be3d7fe
@nickdelgrosso
Merge branch 'main' of https://github.com/ibehave-ibots/course-attend…
0a3c6f7 
…ance-service into nickdelgrosso/issue18
@nickdelgrosso
added approach to getting all workshops from a group
adc506e
@nickdelgrosso
cleaner adapter approach between repo and api, better test of group u…
6252ef4 
…sers listing
@nickdelgrosso
split out tests and api into own modules
2a6710c
@nickdelgrosso
removed unused workflow
420e6f8
@nickdelgrosso
changed workflow to show_workshop_plan()
bd41081
@nickdelgrosso
start on past workshops
e9b2783
@nickdelgrosso
removed new workshop workflows, just merged them again
d12d5ac
@nickdelgrosso nickdelgrosso self-assigned this Jul 27, 2023
@nickdelgrosso nickdelgrosso marked this pull request as ready for review July 27, 2023 21:14
@nickdelgrosso nickdelgrosso changed the title Created a basic GetWorkshopsWorfklows class Create Course Certificate Jul 29, 2023
@nickdelgrosso nickdelgrosso marked this pull request as draft July 29, 2023 10:59
@nickdelgrosso
added workshop topics for certificate
64d6db6
@nickdelgrosso
added organizer to certificate, reorganized text
03ff009
@nickdelgrosso
added end-to-end test for certificate
e913981
@nickdelgrosso
removed loop from predetermined test case
ab1b51a
@nickdelgrosso
split workflows into seperate sections, partially-added a certificate…
5a6d478 
… saving step to the workflow
@nickdelgrosso
more-descriptive filenames, to distinguish repo types
23258c9
@nickdelgrosso
extended workflow integration test out to filesystem
a0f2844
@nickdelgrosso
R: nifty format string date syntax
62f4933
@nickdelgrosso
file renaming: put _test on end of name, to group related modules alp…
a878054 
…habetically
@nickdelgrosso
removed singledispatch approach to filesystem api
a7eeb3c
@nickdelgrosso
restructured files to better-reveal arachitecture
110de48
@nickdelgrosso
split core into interfaces to be directly imported, small renamings
9022509
@nickdelgrosso
imported annotation (code smell reveal: architecture break)
191da82
@nickdelgrosso
split modules into architecture-named subpackages
bd6a601
@nickdelgrosso
split interface into own module in core
03f90bc
@nickdelgrosso nickdelgrosso marked this pull request as ready for review July 31, 2023 08:52
@mohammadbashiri mohammadbashiri merged commit 6eb610c into main Aug 1, 2023
2 checks passed
@mohammadbashiri mohammadbashiri deleted the nickdelgrosso/issue18 branch August 1, 2023 14:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@nickdelgrosso nickdelgrosso

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Make Course Certificates for a Given Workshop
2 participants
@nickdelgrosso @mohammadbashiri