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

Fixed the url for INSPIRE workshop zip file #330

Merged
merged 1 commit into from
Jan 17, 2024

Conversation

ylyangtw
Copy link
Contributor

After release, this url for inspire workshop zip file should be available: https://github.com/IN-CORE/incore-docs/blob/main/workshops/20231115/workshop_2023-11-15.zip

No test is needed. Please check the changes are ok.

@ylyangtw ylyangtw linked an issue Jan 17, 2024 that may be closed by this pull request
@ylyangtw ylyangtw changed the title Fixed the url Fixed the url for INSPIRE workshop zip file Jan 17, 2024
@ylyangtw ylyangtw self-assigned this Jan 17, 2024
@ylyangtw ylyangtw requested a review from a team January 17, 2024 15:36
@longshuicy
Copy link
Member

Ah actually can you change the other way around? then you don't need to rename the zip :-)
Just change the url to point to https://github.com/IN-CORE/incore-docs/blob/main/workshops/20231115/workshop-2023-11-15.zip?

Copy link
Member

@ywkim312 ywkim312 left a comment

Choose a reason for hiding this comment

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

zip file and contents inside it looked good to me. Approve.

@ylyangtw
Copy link
Contributor Author

ylyangtw commented Jan 17, 2024

workshop-2023-11-15.zip

Yeah I thought about that too. But I changed my mind because some notebooks are using workshop_2023-11-15.zip, also workshops 20210526 and 20220131 are using workshop_xxxx-xx-xx.zip as naming.
image

@longshuicy
Copy link
Member

workshop-2023-11-15.zip

Yeah I thought about that too. But I changed my mind because some notebooks are using workshop_2023-11-15.zip, also workshops 20210526 and 20220131 are using workshop_xxxx-xx-xx.zip as naming. image

Make sense. I'll approve.

@longshuicy longshuicy merged commit afda1a3 into develop Jan 17, 2024
1 check passed
@navarroc navarroc deleted the 329-inspire-workshop-zip-file-is-not-available branch February 2, 2024 15:11
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.

Inspire workshop zip file is not available
3 participants