Skip to content
This repository was archived by the owner on Jun 4, 2023. It is now read-only.

Remove API dependency#3

Closed
rodikh wants to merge 24 commits intomasterfrom
v1.0
Closed

Remove API dependency#3
rodikh wants to merge 24 commits intomasterfrom
v1.0

Conversation

@rodikh
Copy link
Copy Markdown
Contributor

@rodikh rodikh commented Mar 31, 2021

davbree and others added 20 commits March 9, 2021 09:34
- added sanity logic
- added datocms
- updated sanity token param
- updated package versions
- updated sanity npx bin
- updated packages to public
- added executable header to executable files
- added lodash to deps
- added bin to stackbit-pull
- added bin to datocms and contentful
- fixed datocms options
- Updated Readme files
Comment thread lerna.json Outdated
"packages": [
"packages/*"
],
"version": "1.0.1-alpha.7"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@rodikh I think we should avoid releasing 1.x.x version.
Let's stick with minor for now.
The last one was 0.4.x. So we can use 0.5.x with "5" serving as non backward-compatible major.

Plus, I wish to avoid major "1" because we want to move to sourcebit anyway.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

also, set version to "independent"

Comment thread lerna.json Outdated
"packages": [
"packages/*"
],
"version": "1.0.1-alpha.7"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

also, set version to "independent"

rodikh added 4 commits April 4, 2021 20:52
- independant versioning
- downgraded core to 0.5
- downgraded all versions to 0.5
- downgraded all versions to 0.5
 - @stackbit/contentful-pull@0.5.0-alpha.0
 - @stackbit/stackbit-pull-core@0.5.0-alpha.0
 - @stackbit/datocms-pull@0.5.0-alpha.0
 - @stackbit/devto-pull@0.5.0-alpha.0
 - @stackbit/sanity-pull@0.5.0-alpha.0
 - @stackbit/stackbit-pull@0.5.0-alpha.0
Copy link
Copy Markdown
Contributor

@smnh smnh left a comment

Choose a reason for hiding this comment

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

@rodikh please merge this one #2 into your pull request as well. Otherwise it won't be mergeable as you removed the original files

* @param options {{dataset: string, preview: boolean, resolveLinks: boolean, allObjects: boolean, dataFormat: string, metadata: boolean, studioUrl: string}}
* @returns {PromiseLike<T>}
*/
function pull(projectId, ssgType, token, options) {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

It seems that you have renamed the standard environment variable to dataset.
Will the container continue to work if parameters are different?

I think container's options have to be supported in order for it to work with the new stackbit-pull.
Currently container sends the following options.

const pullOptions = _.omitBy({
  stackbitPullApiUrl: pullUrl,
  apiKey: STACKBIT_API_SECRET,
  dataFormat: 'object',
  ver: 2,
  preview: true,
  metadata: true,
  resolveLinks: false,
  environment: this.repoPublishBranch || this.repoBranch
}, _.isNil);

So either stackbit-pull needs additional layer that translates the parameters, or every specific cms pull (contentful-pull/sanity-pull) needs to support common option names such as preview/metadata/resolveLinks/environments

@smnh smnh closed this Apr 4, 2021
@smnh smnh deleted the v1.0 branch April 4, 2021 19:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants