Skip to content

Support for Config #34

Description

@natanrolnik

One of my favorite features of Parse is Config. Well, we definitely can create a custom method, that returns a JSON and etc., but would be amazing to have it working without the need to change code in the Client side.

What are the reasons that cause Config not to be supported currently? Maybe answering this question will give some direction on how to implement it?

Activity

  1. flessard commented on Jan 29, 2016

    @flessard
    Contributor

    +1

  2. kingmatusevich commented on Jan 29, 2016

    @kingmatusevich

    +1

  3. AdityaShirole commented on Jan 29, 2016

    @AdityaShirole

    +1

  4. gfosco commented on Jan 29, 2016

    @gfosco
    Contributor

    The two of us building this didn't put a high value on Config, but we totally understand it is useful. I think adding support for it will be pretty simple. I'd venture that this would be a good first PR for someone learning the project.

  5. natanrolnik commented on Jan 30, 2016

    @natanrolnik
    ContributorAuthor

    Hey @gfosco, great idea!

    I took a look on the SDK to understand what is the endpoint - PFRESTConfigCommand generates a GET config. Also, noticed that it saves the last config in the MongoDB.

    What would be, in your opinion, the correct approach to this?

    First, how would we add new parameters or edit existing parameters? Only via REST API, or through some UI similar to what we have today?

    Secondly, how do you suggest creating the class that will handle the GET request?

  6. gfosco commented on Feb 2, 2016

    @gfosco
    Contributor

    Parse Config only hosts a single endpoint for clients, a GET on /config

    Check analytics.js and installations.js for example route handling.

    Sadly, we named something config.js.... So, we either have to rename config.js which would not be fun, or perhaps handle the /config route in index.js.

    Then, we need a way to actually set the config values... Perhaps a POST against /config with the master key. And we should cache the values...

  7. theill commented on Feb 7, 2016

    @theill
    Contributor

    Gave it a first shot with #283

  8. gfosco commented on Feb 14, 2016

    @gfosco
    Contributor

    Merged in #283. 👍

  9. added a commit that references this issue on Aug 27, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions